commit 8d1203d6c57e5d7edf39874233aee92870b8c3bd
author David Benjamin <davidben@google.com> Tue Jul 17 00:38:32 2018 -0400
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Tue Jul 17 20:01:26 2018 +0000
tree 04310caab3c0747b5f1836a5f9b758dafe0805fc
parent 861abccb95d522127a61b48fc565efd6e7b7e68b

Fix some malloc error handling.

Thanks to Tom Thorogood for catching this.

Change-Id: I09fa5d9822b9ba13b106add251e26c6ebee21b03
Reviewed-on: https://boringssl-review.googlesource.com/29825
Reviewed-by: Tom Thorogood <me+google@tomthorogood.co.uk>
Reviewed-by: Steven Valdez <svaldez@chromium.org>
Reviewed-by: David Benjamin <davidben@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>

ssl/ssl_lib.cc

diff --git a/ssl/ssl_lib.cc b/ssl/ssl_lib.cc
index 52f1ccd..9084610 100644
--- a/ssl/ssl_lib.cc
+++ b/ssl/ssl_lib.cc
@@ -2057,8 +2057,8 @@
   if (!PushToStack(ctx->cert_compression_algs.get(), std::move(alg))) {
     if (sk_CertCompressionAlg_num(ctx->cert_compression_algs.get()) == 0) {
       ctx->cert_compression_algs.reset();
-      return 0;
     }
+    return 0;
   }
 
   return 1;