Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20055 )
Change subject: [security] handle BIO_new ...................................................................... [security] handle BIO_new This patch adds verification of the result returned by the OpenSSL's BIO_new() function. It's a rare case when BIO_new() returns nullptr, but if it does so, let's handle the results as documented. Change-Id: I7a18034fde219c181f13d2d4c1ee9acb7e8e1a46 Reviewed-on: http://gerrit.cloudera.org:8080/19819 Tested-by: Kudu Jenkins Reviewed-by: Yuqi Du <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit a7684eacf26bf84c415b8bb32228d026b639f7a8) Conflicts: src/kudu/util/openssl_util_bio.h Reviewed-on: http://gerrit.cloudera.org:8080/20055 Reviewed-by: Yingchun Lai <[email protected]> --- M src/kudu/security/cert.cc M src/kudu/security/cert.h M src/kudu/security/crypto.cc M src/kudu/security/tls_handshake.cc M src/kudu/util/openssl_util.h M src/kudu/util/openssl_util_bio.h 6 files changed, 15 insertions(+), 13 deletions(-) Approvals: Kudu Jenkins: Verified Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20055 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I7a18034fde219c181f13d2d4c1ee9acb7e8e1a46 Gerrit-Change-Number: 20055 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]>
