Merge pull request #10027 from slava77/CMSSW_7_4_6_patch2/t0-mt-crash…

…070115/fix-9282-bp Fix use-after-free in DynamicTruncation::preliminaryFit (backport of 75X #9282 )
cms-sw · Jul 4, 2015 · e4cfafa · e4cfafa
2 parents 9f51b1d + 43582b4
commit e4cfafa
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/RecoMuon/GlobalTrackingTools/src/DynamicTruncation.cc b/RecoMuon/GlobalTrackingTools/src/DynamicTruncation.cc
@@ -388,7 +388,7 @@ void DynamicTruncation::preliminaryFit(map<int, vector<DetId> > compatibleIds, m
     }
   }
   if (!prelFitMeas.empty()) prelFitMeas.pop_back();
-  for (ConstRecHitContainer::const_iterator imrh = prelFitMeas.end(); imrh != prelFitMeas.begin(); imrh-- ) {
+  for (auto imrh = prelFitMeas.rbegin(); imrh != prelFitMeas.rend(); ++imrh) {
     DetId id = (*imrh)->geographicalId(); 
     TrajectoryStateOnSurface tmp = propagatorPF->propagate(prelFitState, theG->idToDet(id)->surface());
     if (tmp.isValid()) prelFitState = tmp;