From e8acf6463aa1102431db5323ee2a1d0f151f50e1 Mon Sep 17 00:00:00 2001
From: Steven Schramm <steven.schramm@cern.ch>
Date: Wed, 12 Sep 2018 08:40:42 +0200
Subject: [PATCH] Fix small memory leak
One of the sub-tools of the jet calibration tool was not properly
deleted, so multiple tool usage led to a small memory leak. This
has been fixed.
Former-commit-id: 9f56f66a9e0cb5d3e0b725b924721e18829af02c
---
Reconstruction/Jet/JetCalibTools/Root/JetCalibrationTool.cxx | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Reconstruction/Jet/JetCalibTools/Root/JetCalibrationTool.cxx b/Reconstruction/Jet/JetCalibTools/Root/JetCalibrationTool.cxx
index 390afe7c2b57..2f996d34ebd2 100644
--- a/Reconstruction/Jet/JetCalibTools/Root/JetCalibrationTool.cxx
+++ b/Reconstruction/Jet/JetCalibTools/Root/JetCalibrationTool.cxx
@@ -21,7 +21,7 @@ JetCalibrationTool::JetCalibrationTool(const std::string& name)
: JetCalibrationToolBase::JetCalibrationToolBase( name ),
m_jetAlgo(""), m_config(""), m_calibSeq(""), m_calibAreaTag(""), m_originScale(""), m_devMode(false), m_isData(true), m_timeDependentCalib(false), m_rhoKey("auto"), m_dir(""), m_eInfoName(""), m_globalConfig(NULL),
m_doJetArea(true), m_doResidual(true), m_doOrigin(true), m_doGSC(true),
- m_jetPileupCorr(NULL), m_etaJESCorr(NULL), m_globalSequentialCorr(NULL), m_insituDataCorr(NULL), m_jetMassCorr(NULL)
+ m_jetPileupCorr(NULL), m_etaJESCorr(NULL), m_globalSequentialCorr(NULL), m_insituDataCorr(NULL), m_jetMassCorr(NULL), m_jetSmearCorr(NULL)
{
declareProperty( "JetCollection", m_jetAlgo = "AntiKt4LCTopo" );
@@ -48,6 +48,7 @@ JetCalibrationTool::~JetCalibrationTool() {
if (m_globalSequentialCorr) delete m_globalSequentialCorr;
if (m_insituDataCorr) delete m_insituDataCorr;
if (m_jetMassCorr) delete m_jetMassCorr;
+ if (m_jetSmearCorr) delete m_jetSmearCorr;
}
--
GitLab