Fix two 1-element memory leaks, and don't create an unused ovStoreHis…

…togram object.
sgblanch · Sep 8, 2017 · e88ad47 · e88ad47
1 parent f8f55b0
commit e88ad47
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/stores/ovStoreBuild.C b/src/stores/ovStoreBuild.C
@@ -308,7 +308,8 @@ computeIIDperBucket(uint32          fileLimit,
           olapsPerBucketMax * GBperOlap + MEMORY_OVERHEAD / 1024.0 / 1024.0 / 1024.0);
   fprintf(stderr, "\n");
 
-  delete hist;
+  delete [] oPR;
+  delete    hist;
 
   return(iidToBucket);
 }
@@ -540,6 +541,8 @@ main(int argc, char **argv) {
     delete inputFile;
   }
 
+  delete [] iidToBucket;
+
   for (uint32 i=0; i<dumpFileMax; i++)
     delete dumpFile[i];
 
@@ -579,8 +582,6 @@ main(int argc, char **argv) {
     if (dumpLengthMax < dumpLength[i])
       dumpLengthMax = dumpLength[i];
 
-  ovStoreHistogram   *histogram = new ovStoreHistogram;
-
   ovOverlap  *overlapsort = ovOverlap::allocateOverlaps(gkp, dumpLengthMax);
 
   for (uint32 i=0; i<dumpFileMax; i++) {