Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8334 )
Change subject: IMPALA-6060: Check the return value of JNI exception handling functions ...................................................................... IMPALA-6060: Check the return value of JNI exception handling functions When JVM runs out of memory and throws an error to JNI, the error handling code uses JNI to get the exception message, resulting in a null pointer and crashing the process. This patch adds error handling code to JniUtil::GetJniExceptionMsg(). Change-Id: Ie3ed88bf8739c56a066f2402727c8204e96aa116 Reviewed-on: http://gerrit.cloudera.org:8080/8334 Reviewed-by: Alex Behm <alex.b...@cloudera.com> Tested-by: Impala Public Jenkins --- M be/src/util/jni-util.cc M be/src/util/jni-util.h M be/src/util/logging-support.cc M fe/src/main/java/org/apache/impala/common/JniUtil.java 4 files changed, 30 insertions(+), 18 deletions(-) Approvals: Alex Behm: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8334 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie3ed88bf8739c56a066f2402727c8204e96aa116 Gerrit-Change-Number: 8334 Gerrit-PatchSet: 5 Gerrit-Owner: Tianyi Wang <tw...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com>