commit b2d987b47c7e87477aa4230bf7b237ea79957c1e
author David Benjamin <davidben@chromium.org> Thu Aug 06 17:09:38 2015 -0400
committer Adam Langley <agl@google.com> Fri Aug 07 02:00:35 2015 +0000
tree 25ffc53b44d4136df5cfd76b29c557d56168d802
parent ade291c5666b12af878fdb6bd9b8d4c1263c868d

Free tmp_wNAF in all exit paths.

BUG=517495

Change-Id: I67c9c511d4ed558ab7a976d6708398e02194cd0f
Reviewed-on: https://boringssl-review.googlesource.com/5628
Reviewed-by: Adam Langley <agl@google.com>

crypto/ec/wnaf.c

diff --git a/crypto/ec/wnaf.c b/crypto/ec/wnaf.c
index a58e107..7fa0e1b 100644
--- a/crypto/ec/wnaf.c
+++ b/crypto/ec/wnaf.c
@@ -464,6 +464,7 @@
           numblocks = (tmp_len + blocksize - 1) / blocksize;
           if (numblocks > pre_comp->numblocks) {
             OPENSSL_PUT_ERROR(EC, ERR_R_INTERNAL_ERROR);
+            OPENSSL_free(tmp_wNAF);
             goto err;
           }
           totalnum = num + numblocks;
@@ -478,6 +479,7 @@
             wNAF_len[i] = blocksize;
             if (tmp_len < blocksize) {
               OPENSSL_PUT_ERROR(EC, ERR_R_INTERNAL_ERROR);
+              OPENSSL_free(tmp_wNAF);
               goto err;
             }
             tmp_len -= blocksize;