Merge pull request #39895 from perrotta/dealWithMissingFiles

Deal with cases in which file is a nullptr in DQMRootSource.cc
cms-sw · Nov 2, 2022 · 8801a99 · 8801a99
2 parents c9c6459 + ecb7f8c
commit 8801a99
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/DQMServices/FwkIO/plugins/DQMRootSource.cc b/DQMServices/FwkIO/plugins/DQMRootSource.cc
@@ -491,7 +491,7 @@ std::shared_ptr<edm::FileBlock> DQMRootSource::readFile_() {
   m_openFiles.reserve(numFiles);
 
   for (auto& fileitem : m_catalog.fileCatalogItems()) {
-    TFile* file;
+    TFile* file = nullptr;
     std::string pfn;
     std::string lfn;
     std::list<std::string> exInfo;
@@ -556,7 +556,7 @@ std::shared_ptr<edm::FileBlock> DQMRootSource::readFile_() {
       }
     }  //end loop over names of the file
 
-    if (!isGoodFile && m_skipBadFiles)
+    if (!file || (!isGoodFile && m_skipBadFiles))
       continue;
 
     std::unique_ptr<std::string> guid{file->Get<std::string>(kCmsGuid)};