Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16736 )
Change subject: IMPALA-10305 (part 2): Sync Kudu's FIPS compliant changes ...................................................................... IMPALA-10305 (part 2): Sync Kudu's FIPS compliant changes kudu-3210 added FIPS compliant changes for Kudu. In previous patch, we ported the following patches for kudu-3210 into Impala source tree: http://gerrit.cloudera.org:8080/16631/ http://gerrit.cloudera.org:8080/16657/ http://gerrit.cloudera.org:8080/16658/ http://gerrit.cloudera.org:8080/16659/ The last patch http://gerrit.cloudera.org:8080/16659/ fixed an OpenSSL race condition with work around by adding lock before verifying signature. But this issue could be fixed by redefining the thread ID callback without any additional locking. Kudu reverted commit f9f3189a6dbe0636d578d80b1d8e60cf7b2e6686 and added a new patch to redefine the thread ID callback. https://gerrit.cloudera.org/#/c/16730/ https://gerrit.cloudera.org/#/c/16731/ This patch syncs Kudu's code changes of above two patches. Testing: - Passed exhausive tests. Change-Id: I04b9d46b5d7228289565617b8d3cfbef9f3b5ba3 Reviewed-on: http://gerrit.cloudera.org:8080/16736 Reviewed-by: Tim Armstrong <[email protected]> Reviewed-by: Attila Bukor <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/kudu/security/CMakeLists.txt M be/src/kudu/security/crypto.cc M be/src/kudu/security/openssl_util.cc M be/src/kudu/security/openssl_util.h M be/src/kudu/security/tls_context.cc M be/src/kudu/security/tls_handshake.cc 6 files changed, 22 insertions(+), 96 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Attila Bukor: Looks good to me, but someone else must approve Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16736 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I04b9d46b5d7228289565617b8d3cfbef9f3b5ba3 Gerrit-Change-Number: 16736 Gerrit-PatchSet: 2 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
