commit e1b8685770d0e82e5a4a3c5d24ad1602e05f2e83
author David Benjamin <davidben@google.com> Wed Jun 14 14:09:29 2023 -0400
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jun 14 19:36:23 2023 +0000
tree 3df9e33ac55abafe7a7eb73a4d36372e11990385
parent d28237e273b2a2d02358e7de422fb4f23adf3ee5

Log failure to create SSL objects in handshakers

This would have made debugging some cross-version test easier.

Change-Id: I7b1bc160b5acf40ec02b9ed5ac2d836e3203cf9a
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/60805
Commit-Queue: Bob Beck <bbe@google.com>
Reviewed-by: Bob Beck <bbe@google.com>
Auto-Submit: David Benjamin <davidben@google.com>

ssl/test/handshaker.cc

diff --git a/ssl/test/handshaker.cc b/ssl/test/handshaker.cc
index ac89063..72aaaa8 100644
--- a/ssl/test/handshaker.cc
+++ b/ssl/test/handshaker.cc
@@ -61,6 +61,8 @@
   UniquePtr<SSL> ssl =
       config->NewSSL(ctx.get(), /*session=*/nullptr, /*test_state=*/nullptr);
   if (!ssl) {
+    fprintf(stderr, "Error creating SSL object in handshaker.\n");
+    ERR_print_errors_fp(stderr);
     return false;
   }
 
@@ -155,6 +157,8 @@
       config->NewSSL(ctx.get(), /*session=*/nullptr,
                      std::unique_ptr<TestState>(new TestState));
   if (!ssl) {
+    fprintf(stderr, "Error creating SSL object in handshaker.\n");
+    ERR_print_errors_fp(stderr);
     return false;
   }