Merge pull request #35613 from abhih1/FixEcalValidationUBSAN2

Fix out of bounds error in Ecal Validation
cms-sw · Oct 13, 2021 · 9761ed9 · 9761ed9
2 parents 576426e + 3a6ad0c
commit 9761ed9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Validation/EcalDigis/plugins/EcalSelectiveReadoutValidation.cc b/Validation/EcalDigis/plugins/EcalSelectiveReadoutValidation.cc
@@ -446,7 +446,7 @@ void EcalSelectiveReadoutValidation::analyzeEE(const edm::Event& event, const ed
         eventError = true;
         ++nEeZsErrors_;
         pair<int, int> ru = dccCh(frame.id());
-        if (isRuComplete_[ru.first][ru.second - 1])
+        if (isRuComplete_[ru.first - 1][ru.second - 1])
           ++nEeZsErrorsType1_;
         if (nEeZsErrors_ < 3) {
           srApplicationErrorLog_ << event.id() << ", "
@@ -706,7 +706,7 @@ void EcalSelectiveReadoutValidation::analyzeEB(const edm::Event& event, const ed
         eventError = true;
         ++nEbZsErrors_;
         pair<int, int> ru = dccCh(frame.id());
-        if (isRuComplete_[ru.first][ru.second - 1])
+        if (isRuComplete_[ru.first - 1][ru.second - 1])
           ++nEbZsErrorsType1_;
         if (nEbZsErrors_ < 3) {
           srApplicationErrorLog_ << event.id() << ", "