Merge pull request #46202 from iarspider/iarspider-patches-20241002-3

[LLVM Analysis] Fix potential memory leak in ApvAnalysisFactory::constructAuxiliaryApvClasses
cms-sw · Oct 3, 2024 · bbd51d2 · bbd51d2
2 parents 6b4f1d4 + 431b1d4
commit bbd51d2
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/CalibTracker/SiStripAPVAnalysis/src/ApvAnalysisFactory.cc b/CalibTracker/SiStripAPVAnalysis/src/ApvAnalysisFactory.cc
@@ -118,10 +118,12 @@ void ApvAnalysisFactory::constructAuxiliaryApvClasses(ApvAnalysis* theAPV, uint3
       theCM = new MedianCommonModeCalculator();
     } else {
       cout << "Sorry Only Median is available for now, Mean and FastLinear are coming soon" << endl;
+      delete theCommonMode;
       return;
     }
   } else {
     cout << "ApvAnalysisFactory: algorithm " << theAlgorithmType_ << " not supported" << endl;
+    delete theCommonMode;
     return;
   }