Important fix for stack overwrite produced by AliESDtrack::GetIntegra…

…tedTimes which was ignoring the size of the array given as input. Fixed also the callers of this, which should specify the size.

STEER/ESD/AliESDtrack.cxx

@@ -1887,8 +1887,11 @@ void AliESDtrack::GetIntegratedTimes(Double_t *times, Int_t nspec) const
   if(fTrackTime)
     for (int i=nspec; i--;) times[i]=fTrackTime[i];
   else
-    for (int i=AliPID::kSPECIESC; i--;) times[i]=0.0;
-  //
+// The line below is wrong since it does not honor the nspec value
+// The "times" array may have only AliPID::kSPECIES size, as called by:
+// AliESDpidCuts::AcceptTrack()
+//    for (int i=AliPID::kSPECIESC; i--;) times[i]=0.0;
+    for (int i=nspec; i--;) times[i]=0.0;
 }
 //_______________________________________________________________________
 Double_t AliESDtrack::GetIntegratedLength() const{

STEER/STEER/AliGlobalQADataMaker.cxx

@@ -441,7 +441,7 @@ void AliGlobalQADataMaker::MakeESDs(AliESDEvent * event) {
          track->GetDZ(xv,yv,zv,esd->GetMagneticField(),dz);
          if (dz[1]<3.) {
 	   Double_t times[AliPID::kSPECIESC];
-	   track->GetIntegratedTimes(times);
+	   track->GetIntegratedTimes(times, AliPID::kSPECIESC);
 	   Double_t tof=track->GetTOFsignal()/*-847055 -1771207*/;
 	   FillESDsData(kPid2,times[AliPID::kPion]-tof);
 	 }

TOF/TestTOFPID.C

@@ -71,7 +71,7 @@ TestTOFPID(const Char_t *filename, Bool_t calibrateESD = kTRUE, Bool_t correctTE
       /* get TOF time */
       time = track->GetTOFsignal();
       /* get expected times */
-      track->GetIntegratedTimes(timei);
+      track->GetIntegratedTimes(timei, AliPID::kSPECIESC);
 
       /* fill PID histos */
       for (Int_t ipart = 0; ipart < AliPID::kSPECIES; ipart++){