Coverity warnings 23063, 23151, 23152, 23851
authormasera <massimo.masera@cern.ch>
Wed, 14 Jan 2015 14:16:50 +0000 (15:16 +0100)
committermasera <massimo.masera@cern.ch>
Wed, 14 Jan 2015 14:17:18 +0000 (15:17 +0100)
ITS/ITSbase/AliITSInitGeometry.cxx
ITS/ITSbase/AliITSMultReconstructor.cxx
ITS/ITSsim/AliITSv11GeometrySSD.cxx

index effa21f..3cff418 100644 (file)
@@ -821,7 +821,12 @@ void AliITSInitGeometry::DecodeDetectorLayersv11(Int_t mod,Int_t &lay,
     mod2 += kLadPerLayer[lay]*kDetPerLadder[lay];
     lay++;
   } while(mod2<=mod); // end while
-  if(lay>6) Error("DecodeDetectorLayers","lay=%d>6",lay);
+  if(lay>6) {
+    AliError(Form("lay=%d>6 - setting ladder and detector number to zero",lay));
+    lad = 0;
+    det = 0;
+    return;
+  }
 
   mod2 = kLadPerLayer[lay-1]*kDetPerLadder[lay-1] - mod2+mod;
   lad = mod2/kDetPerLadder[lay-1];
index 17ebe5d..3c3142f 100644 (file)
@@ -266,7 +266,7 @@ fhphiClustersLay1(0),
   for (int i=2;i--;) fStoreRefs[i][0] =  fStoreRefs[i][1] = kFALSE;
   TH1::AddDirectory(oldStatus);
 }
-
+/*
 //______________________________________________________________________
 AliITSMultReconstructor::AliITSMultReconstructor(const AliITSMultReconstructor &mr) : 
 AliTrackleter(mr),
@@ -347,6 +347,7 @@ AliITSMultReconstructor& AliITSMultReconstructor::operator=(const AliITSMultReco
   }
   return *this;
 }
+*/
 
 //______________________________________________________________________
 AliITSMultReconstructor::~AliITSMultReconstructor(){
index 748cd97..a04c854 100644 (file)
@@ -1616,9 +1616,8 @@ void AliITSv11GeometrySSD::CreateTransformationMatrices(){
       delete localendladdercarbonfibertrans[i];
   for(Int_t i=0; i<3; i++) delete localladdercablerot[i];
   for(Int_t i=0; i<klocalladdersidecablesnumber; i++){
-       for(Int_t j=0; j<klocalladdercombitransnumber; j++)
-               delete localladdercablecombitransmatrix[i][j];
-               delete []localladdercablecombitransmatrix[i];
+    for(Int_t j=0; j<klocalladdercombitransnumber; j++)delete localladdercablecombitransmatrix[i][j];
+    delete []localladdercablecombitransmatrix[i];
   }
   delete localendladdercliprot;
   delete localendladdercliptrans;