Merge pull request #3 from folguera/smuzaffar_ubsan-l1-emptry-array

Also fix UBSAN error on L989
trackreco · May 31, 2023 · 5393094 · 5393094
2 parents 1334546 + d6cb7fc
commit 5393094
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/L1Trigger/DTTriggerPhase2/src/MuonPathAssociator.cc b/L1Trigger/DTTriggerPhase2/src/MuonPathAssociator.cc
@@ -119,12 +119,12 @@ void MuonPathAssociator::correlateMPaths(edm::Handle<DTDigiCollection> dtdigis,
         bool at_least_one_SL1_confirmation = false;
         bool at_least_one_SL3_confirmation = false;
 
-        bool useFitSL1[SL1metaPrimitives.size()];
+        vector<bool> useFitSL1;
         for (unsigned int i = 0; i < SL1metaPrimitives.size(); i++)
-          useFitSL1[i] = false;
-        bool useFitSL3[SL3metaPrimitives.size()];
+          useFitSL1.push_back(false);
+        vector<bool> useFitSL3;
         for (unsigned int i = 0; i < SL3metaPrimitives.size(); i++)
-          useFitSL3[i] = false;
+          useFitSL3.push_back(false);
 
         //SL1-SL3
         vector<metaPrimitive> chamberMetaPrimitives;
@@ -986,9 +986,9 @@ void MuonPathAssociator::correlateMPaths(edm::Handle<DTDigiCollection> dtdigis,
 }
 
 void MuonPathAssociator::removeSharingFits(vector<metaPrimitive> &chamberMPaths, vector<metaPrimitive> &allMPaths) {
-  bool useFit[chamberMPaths.size()];
+  vector<bool> useFit;
   for (unsigned int i = 0; i < chamberMPaths.size(); i++) {
-    useFit[i] = true;
+    useFit.push_back(true);
   }
   for (unsigned int i = 0; i < chamberMPaths.size(); i++) {
     if (debug_)