Csaba Ringhofer has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21234


Change subject: IMPALA-12969: Release JNI array if DeserializeThriftMsg failed
......................................................................

IMPALA-12969: Release JNI array if DeserializeThriftMsg failed

Before this patch ReleaseByteArrayElements was not called in case
the deserialization failed (e.g. by hitting Thrift's MaxMessageSize).
This could potentially cause JVM/native heap leak, depending on how
the JVM handled the array allocation.

Change-Id: Id2c0335b12e9289ae851d0ec050765951a8ca6c7
---
M be/src/rpc/jni-thrift-util.h
1 file changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/34/21234/1
--
To view, visit http://gerrit.cloudera.org:8080/21234
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id2c0335b12e9289ae851d0ec050765951a8ca6c7
Gerrit-Change-Number: 21234
Gerrit-PatchSet: 1
Gerrit-Owner: Csaba Ringhofer <[email protected]>

Reply via email to