commit 1b0c438e1a0e570de18ecc7aabda3be8dea4cfa0
author Emily Stark <estark@google.com> Sun Mar 06 23:11:31 2016 -0800
committer David Benjamin <davidben@google.com> Mon Mar 07 15:48:21 2016 +0000
tree ecda3cff6e15c67bfd1a735bddc949143be1739f
parent 05c7bb4565b1f099e3aceea3623c073003cf5ccd

Fix i2d_RSAPrivateKey, i2d_RSAPublicKey memory leaks

Change-Id: Id2678c20270f2f45efe56efd65caf23e0bb8c09e
Reviewed-on: https://boringssl-review.googlesource.com/7350
Reviewed-by: David Benjamin <davidben@google.com>

crypto/rsa/rsa_asn1.c

diff --git a/crypto/rsa/rsa_asn1.c b/crypto/rsa/rsa_asn1.c
index c2e93dd..d14ecae 100644
--- a/crypto/rsa/rsa_asn1.c
+++ b/crypto/rsa/rsa_asn1.c
@@ -388,6 +388,7 @@
   CBB cbb;
   if (!CBB_init(&cbb, 0) ||
       !RSA_marshal_public_key(&cbb, in)) {
+    CBB_cleanup(&cbb);
     return -1;
   }
   return CBB_finish_i2d(&cbb, outp);
@@ -415,6 +416,7 @@
   CBB cbb;
   if (!CBB_init(&cbb, 0) ||
       !RSA_marshal_private_key(&cbb, in)) {
+    CBB_cleanup(&cbb);
     return -1;
   }
   return CBB_finish_i2d(&cbb, outp);