diff options
author | David Benjamin <davidben@google.com> | 2024-02-27 15:44:04 -0500 |
---|---|---|
committer | Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2024-03-05 21:19:38 +0000 |
commit | fc1f5213b61edc1da123c36e14ed3f7a70f8bb1b (patch) | |
tree | 5d8f342d1bce612c6502ad2b802f69f32e105cf8 /ssl/ssl_session.cc | |
parent | f4ac688b2bc19a5675dfd9cc17e796a9b8b692af (diff) | |
download | boringssl-fc1f5213b61edc1da123c36e14ed3f7a70f8bb1b.zip boringssl-fc1f5213b61edc1da123c36e14ed3f7a70f8bb1b.tar.gz boringssl-fc1f5213b61edc1da123c36e14ed3f7a70f8bb1b.tar.bz2 |
Align CRYPTO_get_ex_new_index with the public API's calling convention
Although we usually prefer not to use special -1 returns for errors, the
public API does this across the board. Making the internal function
different doesn't do much good.
Change-Id: I6bfe8c9d989da81affeb5cb652de8d3edcbf5efa
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/66649
Reviewed-by: Bob Beck <bbe@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
Diffstat (limited to 'ssl/ssl_session.cc')
-rw-r--r-- | ssl/ssl_session.cc | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/ssl/ssl_session.cc b/ssl/ssl_session.cc index 5275b69..7f9b99f 100644 --- a/ssl/ssl_session.cc +++ b/ssl/ssl_session.cc @@ -1203,12 +1203,7 @@ int SSL_SESSION_get_ex_new_index(long argl, void *argp, CRYPTO_EX_unused *unused, CRYPTO_EX_dup *dup_unused, CRYPTO_EX_free *free_func) { - int index; - if (!CRYPTO_get_ex_new_index(&g_ex_data_class, &index, argl, argp, - free_func)) { - return -1; - } - return index; + return CRYPTO_get_ex_new_index(&g_ex_data_class, argl, argp, free_func); } int SSL_SESSION_set_ex_data(SSL_SESSION *session, int idx, void *arg) { |