Fix memory leaks when calling EVP_PKEY_get1_RSA.

crypto/signature_creator_openssl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "crypto/signature_creator.h"
 
 #include <openssl/evp.h>
 #include <openssl/rsa.h>
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/stl_util.h"
 #include "crypto/openssl_util.h"
 #include "crypto/rsa_private_key.h"
+#include "crypto/scoped_openssl_types.h"
 
 namespace crypto {
 
 // static
 SignatureCreator* SignatureCreator::Create(RSAPrivateKey* key) {
   OpenSSLErrStackTracer err_tracer(FROM_HERE);
   scoped_ptr<SignatureCreator> result(new SignatureCreator);
   result->key_ = key;
   if (!EVP_SignInit_ex(result->sign_context_, EVP_sha1(), NULL))
     return NULL;
   return result.release();
 }
 
 // static
 bool SignatureCreator::Sign(RSAPrivateKey* key,
                             const uint8* data,
                             int data_len,
                             std::vector<uint8>* signature) {
-  RSA* rsa_key = EVP_PKEY_get1_RSA(key->key());
+  ScopedRSA rsa_key(EVP_PKEY_get1_RSA(key->key()));
   if (!rsa_key)
     return false;
-  signature->resize(RSA_size(rsa_key));
+  signature->resize(RSA_size(rsa_key.get()));
 
   unsigned int len = 0;
   bool success = RSA_sign(NID_sha1, data, data_len, vector_as_array(signature),
-                          &len, rsa_key);
+                          &len, rsa_key.get());
   if (!success) {
     signature->clear();
     return false;
   }
   signature->resize(len);
   return true;
 }
 
 SignatureCreator::SignatureCreator()
     : sign_context_(EVP_MD_CTX_create()) {
@@ skipping 17 matching lines @@
   int rv = EVP_SignFinal(sign_context_, vector_as_array(signature), &len, key);
   if (!rv) {
     signature->clear();
     return false;
   }
   signature->resize(len);
   return true;
 }
 
 }  // namespace crypto