Convert one libssl function to C++11. This is to make sure all of libssl's consumers' have sufficiently reasonable toolchains. Once this bakes, we can go about moving libssl to C++. This is just starting with libssl for now because libcrypto has more consumers and libssl would benefit more from C++ than libcrypto (though libcrypto also has code that would benefit). BUG=132 Change-Id: Ie02f7b0a8a95defd289cc7e62451d4b16408ca2a Reviewed-on: https://boringssl-review.googlesource.com/13161 Commit-Queue: David Benjamin <davidben@google.com> Reviewed-by: Adam Langley <agl@google.com> (cherry picked from commit 1252f8758ab1c4c4a14db25f9fa53e5950e4bcda)
diff --git a/ssl/CMakeLists.txt b/ssl/CMakeLists.txt index 9287a63..fae17f1 100644 --- a/ssl/CMakeLists.txt +++ b/ssl/CMakeLists.txt
@@ -25,6 +25,7 @@ ssl_file.c ssl_lib.c ssl_rsa.c + ssl_rsa_cc.cc ssl_session.c ssl_stat.c t1_enc.c
diff --git a/ssl/ssl_rsa.c b/ssl/ssl_rsa.c index e984012..2a4dcbf 100644 --- a/ssl/ssl_rsa.c +++ b/ssl/ssl_rsa.c
@@ -153,18 +153,6 @@ return 1; } -int SSL_use_RSAPrivateKey_ASN1(SSL *ssl, const uint8_t *der, size_t der_len) { - RSA *rsa = RSA_private_key_from_bytes(der, der_len); - if (rsa == NULL) { - OPENSSL_PUT_ERROR(SSL, ERR_R_ASN1_LIB); - return 0; - } - - int ret = SSL_use_RSAPrivateKey(ssl, rsa); - RSA_free(rsa); - return ret; -} - int SSL_use_PrivateKey(SSL *ssl, EVP_PKEY *pkey) { int ret;
diff --git a/ssl/ssl_rsa_cc.cc b/ssl/ssl_rsa_cc.cc new file mode 100644 index 0000000..653308c --- /dev/null +++ b/ssl/ssl_rsa_cc.cc
@@ -0,0 +1,76 @@ +/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) + * All rights reserved. + * + * This package is an SSL implementation written + * by Eric Young (eay@cryptsoft.com). + * The implementation was written so as to conform with Netscapes SSL. + * + * This library is free for commercial and non-commercial use as long as + * the following conditions are aheared to. The following conditions + * apply to all code found in this distribution, be it the RC4, RSA, + * lhash, DES, etc., code; not just the SSL code. The SSL documentation + * included with this distribution is covered by the same copyright terms + * except that the holder is Tim Hudson (tjh@cryptsoft.com). + * + * Copyright remains Eric Young's, and as such any Copyright notices in + * the code are not to be removed. + * If this package is used in a product, Eric Young should be given attribution + * as the author of the parts of the library used. + * This can be in the form of a textual message at program startup or + * in documentation (online or textual) provided with the package. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * 1. Redistributions of source code must retain the copyright + * notice, this list of conditions and the following disclaimer. + * 2. Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * 3. All advertising materials mentioning features or use of this software + * must display the following acknowledgement: + * "This product includes cryptographic software written by + * Eric Young (eay@cryptsoft.com)" + * The word 'cryptographic' can be left out if the rouines from the library + * being used are not cryptographic related :-). + * 4. If you include any Windows specific code (or a derivative thereof) from + * the apps directory (application code) you must include an acknowledgement: + * "This product includes software written by Tim Hudson (tjh@cryptsoft.com)" + * + * THIS SOFTWARE IS PROVIDED BY ERIC YOUNG ``AS IS'' AND + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE + * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF + * SUCH DAMAGE. + * + * The licence and distribution terms for any publically available version or + * derivative of this code cannot be changed. i.e. this code cannot simply be + * copied and put under another distribution licence + * [including the GNU Public Licence.] */ + +#include <openssl/ssl.h> + +#include <openssl/err.h> +#include <openssl/rsa.h> + + +/* This function has been converted to C++ to check if all of libssl's + * consumers' toolchains are capable of handling C++11. Once all problems in + * consumer toolchains are found and fixed, we will convert the rest of + * libssl. */ + +int SSL_use_RSAPrivateKey_ASN1(SSL *ssl, const uint8_t *der, size_t der_len) { + bssl::UniquePtr<RSA> rsa(RSA_private_key_from_bytes(der, der_len)); + if (!rsa) { + OPENSSL_PUT_ERROR(SSL, ERR_R_ASN1_LIB); + return 0; + } + + return SSL_use_RSAPrivateKey(ssl, rsa.get()); +}