Fix use-after-free in DynamicTruncation::preliminaryFit

On the first iteration we used end() of STL container. end() returns
a pointer after the last element in container. In this case, we were
starting iteration from the last __removed__ `TrackingRecHit`.

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>

RecoMuon/GlobalTrackingTools/src/DynamicTruncation.cc

@@ -387,7 +387,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;