From fc7cd46faf5cafc73d4316a21d59cb2b83a6b48f Mon Sep 17 00:00:00 2001 From: Allen Wittenauer Date: Thu, 5 Nov 2015 16:50:55 -0800 Subject: [PATCH] MAPREDUCE-3914. Mismatched free() / delete / delete [] in HadoopPipes (Joe Mudd via aw) --- hadoop-mapreduce-project/CHANGES.txt | 4 ++++ .../hadoop-pipes/src/main/native/pipes/impl/HadoopPipes.cc | 4 ++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/hadoop-mapreduce-project/CHANGES.txt b/hadoop-mapreduce-project/CHANGES.txt index f30f0ef9a93..938509adc45 100644 --- a/hadoop-mapreduce-project/CHANGES.txt +++ b/hadoop-mapreduce-project/CHANGES.txt @@ -120,6 +120,7 @@ Trunk (Unreleased) BUG FIXES + MAPREDUCE-6191. Improve clearing stale state of Java serialization testcase. (Sam Liu via Eric Yang) @@ -168,6 +169,9 @@ Trunk (Unreleased) MAPREDUCE-4695. Fix LocalRunner on trunk after MAPREDUCE-3223 broke it (harsh) + MAPREDUCE-3914. Mismatched free() / delete / delete [] in HadoopPipes + (Joe Mudd via aw) + MAPREDUCE-4574. Fix TotalOrderParitioner to work with non-WritableComparable key types. (harsh) diff --git a/hadoop-tools/hadoop-pipes/src/main/native/pipes/impl/HadoopPipes.cc b/hadoop-tools/hadoop-pipes/src/main/native/pipes/impl/HadoopPipes.cc index 64c845456b4..91fb5a42b23 100644 --- a/hadoop-tools/hadoop-pipes/src/main/native/pipes/impl/HadoopPipes.cc +++ b/hadoop-tools/hadoop-pipes/src/main/native/pipes/impl/HadoopPipes.cc @@ -1169,8 +1169,8 @@ namespace HadoopPipes { if (outStream != NULL) { //fclose(outStream); } - delete bufin; - delete bufout; + delete[] bufin; + delete[] bufout; return true; } catch (Error& err) { fprintf(stderr, "Hadoop Pipes Exception: %s\n",