Merge pull request #6862 from deguio/fixDTatT0

fix DT crash in case of crazy FED number
cms-sw · Dec 12, 2014 · 3f6892d · 3f6892d
2 parents bedcc91 + b4725b0
commit 3f6892d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/DQM/DTMonitorModule/interface/DTDataIntegrityTask.h b/DQM/DTMonitorModule/interface/DTDataIntegrityTask.h
@@ -125,6 +125,10 @@ class DTDataIntegrityTask : public edm::EDAnalyzer {
   std::string outputFile;
   double rob_max[25];
 
+  int FEDIDmin;
+  int FEDIDMax;
+
+
   //Event counter for the graphs VS time
   int myPrevEv;
 

diff --git a/DQM/DTMonitorModule/src/DTDataIntegrityTask.cc b/DQM/DTMonitorModule/src/DTDataIntegrityTask.cc
@@ -861,8 +861,7 @@ void DTDataIntegrityTask::processFED(DTDDUData & data, const std::vector<DTROS25
 
   DTROChainCoding code;
   code.setDDU(ddu);
-
-  code.getDDUID();
+  if(code.getDDUID() < FEDIDmin || code.getDDUID() > FEDIDMax) return;
 
   hFEDEntry->Fill(code.getDDUID());
 
@@ -1236,8 +1235,8 @@ void DTDataIntegrityTask::beginJob() {
 
 
   // Loop over the DT FEDs
-  int FEDIDmin = FEDNumbering::MINDTFEDID;
-  int FEDIDMax = FEDNumbering::MAXDTFEDID;
+  FEDIDmin = FEDNumbering::MINDTFEDID;
+  FEDIDMax = FEDNumbering::MAXDTFEDID;
 
   LogTrace("DTRawToDigi|DTDQM|DTMonitorModule|DTDataIntegrityTask")
     << " FEDS: " << FEDIDmin  << " to " <<  FEDIDMax << " in the RO" << endl;