A bunch more scopers. Change-Id: I5c8dbfec4a404d8d1501725a90b383eb3e05c664 Reviewed-on: https://boringssl-review.googlesource.com/29591 Reviewed-by: Adam Langley <agl@google.com>
diff --git a/ssl/ssl_privkey.cc b/ssl/ssl_privkey.cc index d41ea5d..f5c387b 100644 --- a/ssl/ssl_privkey.cc +++ b/ssl/ssl_privkey.cc
@@ -304,7 +304,7 @@ return 0; } - return ssl_set_pkey(ssl->config->cert, pkey.get()); + return ssl_set_pkey(ssl->config->cert.get(), pkey.get()); } int SSL_use_RSAPrivateKey_ASN1(SSL *ssl, const uint8_t *der, size_t der_len) { @@ -323,7 +323,7 @@ return 0; } - return ssl_set_pkey(ssl->config->cert, pkey); + return ssl_set_pkey(ssl->config->cert.get(), pkey); } int SSL_use_PrivateKey_ASN1(int type, SSL *ssl, const uint8_t *der, @@ -356,7 +356,7 @@ return 0; } - return ssl_set_pkey(ctx->cert, pkey.get()); + return ssl_set_pkey(ctx->cert.get(), pkey.get()); } int SSL_CTX_use_RSAPrivateKey_ASN1(SSL_CTX *ctx, const uint8_t *der, @@ -376,7 +376,7 @@ return 0; } - return ssl_set_pkey(ctx->cert, pkey); + return ssl_set_pkey(ctx->cert.get(), pkey); } int SSL_CTX_use_PrivateKey_ASN1(int type, SSL_CTX *ctx, const uint8_t *der, @@ -461,21 +461,6 @@ return alg != nullptr && alg->is_rsa_pss; } -static int set_algorithm_prefs(uint16_t **out_prefs, size_t *out_num_prefs, - const uint16_t *prefs, size_t num_prefs) { - OPENSSL_free(*out_prefs); - - *out_num_prefs = 0; - *out_prefs = (uint16_t *)BUF_memdup(prefs, num_prefs * sizeof(prefs[0])); - if (*out_prefs == NULL) { - OPENSSL_PUT_ERROR(SSL, ERR_R_MALLOC_FAILURE); - return 0; - } - *out_num_prefs = num_prefs; - - return 1; -} - int SSL_CTX_set_signing_algorithm_prefs(SSL_CTX *ctx, const uint16_t *prefs, size_t num_prefs) { return ctx->cert->sigalgs.CopyFrom(MakeConstSpan(prefs, num_prefs)); @@ -491,6 +476,5 @@ int SSL_CTX_set_verify_algorithm_prefs(SSL_CTX *ctx, const uint16_t *prefs, size_t num_prefs) { - return set_algorithm_prefs(&ctx->verify_sigalgs, &ctx->num_verify_sigalgs, - prefs, num_prefs); + return ctx->verify_sigalgs.CopyFrom(MakeConstSpan(prefs, num_prefs)); }