commit b7c5e84847f6b92aee8d5825cd46d316db318a00
author David Benjamin <davidben@google.com> Mon Mar 28 09:59:10 2016 -0400
committer David Benjamin <davidben@google.com> Mon Mar 28 17:17:32 2016 +0000
tree 0cce01a6ece4413375473ca40bdef8d0b6eb63c8
parent e29ea166a6ec44bbfce81d1c93f064b9aca0f443

Fix some malloc test failures.

These only affect the tests.

Change-Id: If22d047dc98023501c771787b485276ece92d4a2
Reviewed-on: https://boringssl-review.googlesource.com/7573
Reviewed-by: Steven Valdez <svaldez@google.com>
Reviewed-by: David Benjamin <davidben@google.com>

crypto/x509/x509_test.cc

diff --git a/crypto/x509/x509_test.cc b/crypto/x509/x509_test.cc
index 1fce619..650163a 100644
--- a/crypto/x509/x509_test.cc
+++ b/crypto/x509/x509_test.cc
@@ -243,6 +243,9 @@
 // bumping the reference counts for each certificate in question.
 static STACK_OF(X509)* CertsToStack(const std::vector<X509*> &certs) {
   ScopedX509Stack stack(sk_X509_new_null());
+  if (!stack) {
+    return nullptr;
+  }
   for (auto cert : certs) {
     if (!sk_X509_push(stack.get(), cert)) {
       return nullptr;