Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/19292


Change subject: [jwt] proper memory ownership for RSA key components
......................................................................

[jwt] proper memory ownership for RSA key components

This patch corrects the ownership for RSA key's module and exponent in
RSAJWTPublicKeyBuilder::ConvertJwkToPem() for OpenSSL versions prior
to 1.1.0.  Prior to this patch, jwt-util-test would crash with SIGSEGV
if compiled and run against earlier OpenSSL versions (e.g. 1.0.1)

*** SIGSEGV (@0x1e800002) received by PID 31969 (TID 0x7f07a0ae5940) from PID 
511705090; stack trace: ***
    ...
    @       0x3ae366ad5d CRYPTO_free at ??:0
    @       0x3ae36a6ef5 BN_free at ??:0
    @     0x7f07a9a44d85 std::_Function_handler<>::_M_invoke() at ??:0
    @     0x7f07a9a4571d std::function<>::operator()() at ??:0
    @     0x7f07a99f42ac std::unique_ptr<>::~unique_ptr() at ??:0
    @     0x7f07a99e6cfa kudu::RSAJWTPublicKeyBuilder::ConvertJwkToPem() at ??:0
    @     0x7f07a99e5910 kudu::RSAJWTPublicKeyBuilder::CreateJWKPublicKey() at 
??:0
    ...

Change-Id: Ib94122eb7221c99ed4997b456cdcea4cd6d5154b
---
M src/kudu/util/jwt-util.cc
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/92/19292/1
--
To view, visit http://gerrit.cloudera.org:8080/19292
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib94122eb7221c99ed4997b456cdcea4cd6d5154b
Gerrit-Change-Number: 19292
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to