From 612c71638c04593f3938781ac5f7437c16d707a8 Mon Sep 17 00:00:00 2001 From: scott snyder Date: Mon, 29 Aug 2022 21:52:51 +0200 Subject: [PATCH] LArCalibUtils: Fix gcc12 warning. Potential null pointer dereference. --- LArCalorimeter/LArCalibUtils/src/LArOFCAlg.cxx | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/LArCalorimeter/LArCalibUtils/src/LArOFCAlg.cxx b/LArCalorimeter/LArCalibUtils/src/LArOFCAlg.cxx index 23815d0c52f..9ef9be59bfd 100644 --- a/LArCalorimeter/LArCalibUtils/src/LArOFCAlg.cxx +++ b/LArCalorimeter/LArCalibUtils/src/LArOFCAlg.cxx @@ -1,7 +1,7 @@ //Dear emacs, this is -*- c++ -*- /* - Copyright (C) 2002-2021 CERN for the benefit of the ATLAS collaboration + Copyright (C) 2002-2022 CERN for the benefit of the ATLAS collaboration */ // LArOFC: Algorithm to calculate optimal filtering constants. @@ -354,6 +354,11 @@ StatusCode LArOFCAlg::stop() if (nFailed) ATH_MSG_ERROR( "Number of channels * gains with failed OFC verification: " << nFailed ); + if ( m_dumpOFCfile.size()) { + ATH_MSG_INFO( "Dumping OFCs to file " << m_dumpOFCfile ) ; + larOFCComplete->dumpOFC(m_dumpOFCfile) ; + } + // record and symlink LArOFCComplete object LArOFCComplete* larOFCCompletePtr=larOFCComplete.get(); sc = detStore()->record(std::move(larOFCComplete),m_ofcKey); @@ -389,11 +394,6 @@ StatusCode LArOFCAlg::stop() ATH_MSG_INFO( "Symlink with ILArOFC done" ) ; } - if ( m_dumpOFCfile.size()) { - ATH_MSG_INFO( "Dumping OFCs to file " << m_dumpOFCfile ) ; - larOFCComplete->dumpOFC(m_dumpOFCfile) ; - } - if (larOFCBinComplete) { sc = detStore()->record(std::move(larOFCBinComplete),m_ofcBinKey); if (sc.isFailure()) { -- GitLab