You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by so...@apache.org on 2020/08/24 11:01:00 UTC
[hadoop] branch trunk updated: HADOOP-17209. Erasure Coding: Native
library memory leak. Contriubted by Sean Chow
This is an automated email from the ASF dual-hosted git repository.
sodonnell pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/trunk by this push:
new 17cd8a1 HADOOP-17209. Erasure Coding: Native library memory leak. Contriubted by Sean Chow
17cd8a1 is described below
commit 17cd8a1b1627f4d87ddc5dcc2ec7c738ffdc576b
Author: S O'Donnell <so...@cloudera.com>
AuthorDate: Mon Aug 24 11:24:31 2020 +0100
HADOOP-17209. Erasure Coding: Native library memory leak. Contriubted by Sean Chow
---
.../src/main/native/src/org/apache/hadoop/io/erasurecode/jni_common.c | 4 +++-
.../main/native/src/org/apache/hadoop/io/erasurecode/jni_rs_decoder.c | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_common.c b/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_common.c
index 9cca6dd..816536b 100644
--- a/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_common.c
+++ b/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_common.c
@@ -92,6 +92,7 @@ void getInputs(JNIEnv *env, jobjectArray inputs, jintArray inputOffsets,
destInputs[i] = NULL;
}
}
+ (*env)->ReleaseIntArrayElements(env, inputOffsets, tmpInputOffsets, 0);
}
void getOutputs(JNIEnv *env, jobjectArray outputs, jintArray outputOffsets,
@@ -112,4 +113,5 @@ void getOutputs(JNIEnv *env, jobjectArray outputs, jintArray outputOffsets,
byteBuffer));
destOutputs[i] += tmpOutputOffsets[i];
}
-}
\ No newline at end of file
+ (*env)->ReleaseIntArrayElements(env, outputOffsets, tmpOutputOffsets, 0);
+}
diff --git a/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_rs_decoder.c b/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_rs_decoder.c
index 52d255a..72314d2 100644
--- a/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_rs_decoder.c
+++ b/hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/io/erasurecode/jni_rs_decoder.c
@@ -66,6 +66,7 @@ jintArray outputOffsets) {
decode(&rsDecoder->decoder, rsDecoder->inputs, tmpErasedIndexes,
numErased, rsDecoder->outputs, chunkSize);
+ (*env)->ReleaseIntArrayElements(env, erasedIndexes, tmpErasedIndexes, 0);
}
JNIEXPORT void JNICALL
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org