Fixes for Coverity warnings
authormasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 31 Jan 2011 17:12:42 +0000 (17:12 +0000)
committermasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 31 Jan 2011 17:12:42 +0000 (17:12 +0000)
ITS/AliITSVertexer3DTapan.cxx
ITS/AliITStrackerMI.cxx
ITS/AliITStrackerSA.cxx
ITS/AliITStrackerSA.h
ITS/AliITStrackerV2.cxx

index 57b20d9..88ac10c 100644 (file)
@@ -54,7 +54,7 @@ void AliITSVertexer3DTapan::LoadClusters(TTree *cTree) {
        if (lay>2) break;  //load the SPD clusters only
 
        Int_t ncl=clusters->GetEntriesFast();
-       Float_t hPhi;
+       Float_t hPhi=0.;
        while (ncl--) {
           AliITSRecPoint *c=(AliITSRecPoint*)clusters->UncheckedAt(ncl);
          Float_t pos[3];
@@ -89,7 +89,8 @@ AliESDVertex *AliITSVertexer3DTapan::FindVertexForCurrentEvent(TTree *cTree) {
   //
   LoadClusters(cTree);
 
-  Double_t pos[3], postemp[3], sigpos[3];
+  Double_t pos[3], postemp[3];
+  Double_t sigpos[3]={0.,0.,0.};
   Int_t ncontr, ncontrtemp;
   Float_t cuts[3];
   Int_t vtxstatus=0;
@@ -138,7 +139,11 @@ AliESDVertex *AliITSVertexer3DTapan::FindVertexForCurrentEvent(TTree *cTree) {
   }
   AliInfo(Form("Final step: %d %f %f %f st=%d",ncontr,pos[0],pos[1],pos[2],vtxstatus));
 
-  return new AliESDVertex(pos,sigpos,(Double_t)vtxstatus,ncontr,"AliITSVertexer3DTapan");
+  Double_t covma[6]={0.,0.,0.,0.,0.,0.};
+  covma[0]=sigpos[0];
+  covma[2]=sigpos[1];
+  covma[5]=sigpos[2];
+  return new AliESDVertex(pos,covma,(Double_t)vtxstatus,ncontr,"AliITSVertexer3DTapan");
 
 }
 
index 75e423d..7203125 100644 (file)
@@ -92,9 +92,16 @@ fPlaneEff(0) {
   //Default constructor
   Int_t i;
   for(i=0;i<4;i++) fSPDdetzcentre[i]=0.;
-  for(i=0;i<2;i++) {fxOverX0Shield[i]=-1.;fxTimesRhoShield[i]=-1.;}
+  for(i=0;i<2;i++) {
+    fxOverX0Shield[i]=-1.;
+    fxTimesRhoShield[i]=-1.;
+    fConstraint[i]=0;
+  }
   for(i=0;i<6;i++) {fxOverX0Layer[i]=-1.;fxTimesRhoLayer[i]=-1.;}
   fOriginal.SetOwner();
+  for(i=0;i<AliITSgeomTGeo::kNLayers;i++)fForceSkippingOfLayer[i]=0;
+  for(i=0;i<100000;i++)fBestTrackIndex[i]=0;
+
 }
 //------------------------------------------------------------------------
 AliITStrackerMI::AliITStrackerMI(const Char_t *geom) : AliTracker(),
@@ -256,6 +263,7 @@ fPlaneEff(0) {
     if(AliITSReconstructor::GetRecoParam()->GetHistoPlaneEff()) fPlaneEff->SetCreateHistos(kTRUE);
   }
 }
+/*
 //------------------------------------------------------------------------
 AliITStrackerMI::AliITStrackerMI(const AliITStrackerMI &tracker):AliTracker(tracker),
 fI(tracker.fI),
@@ -300,6 +308,7 @@ fPlaneEff(tracker.fPlaneEff) {
     fxTimesRhoShield[i]=tracker.fxTimesRhoShield[i];
   }
 }
+
 //------------------------------------------------------------------------
 AliITStrackerMI & AliITStrackerMI::operator=(const AliITStrackerMI &tracker){
   //Assignment operator
@@ -307,6 +316,7 @@ AliITStrackerMI & AliITStrackerMI::operator=(const AliITStrackerMI &tracker){
   new(this) AliITStrackerMI(tracker);
   return *this;
 }
+*/
 //------------------------------------------------------------------------
 AliITStrackerMI::~AliITStrackerMI()
 {
@@ -1499,7 +1509,7 @@ fNMaxSigmaCl(3)
       fBy20[j1][1]=0;
     }
   }
-
+  for(Int_t i=0;i<AliITSRecoParam::fgkMaxClusterPerLayer;i++)fClusters[i]=NULL;
 
 }
 //------------------------------------------------------------------------
@@ -1587,8 +1597,9 @@ fNMaxSigmaCl(3) {
       fBy20[j1][1]=0;
     }
   }
-
+  for(Int_t i=0;i<AliITSRecoParam::fgkMaxClusterPerLayer;i++)fClusters[i]=NULL;
 }
+/*
 //------------------------------------------------------------------------
 AliITStrackerMI::AliITSlayer::AliITSlayer(const AliITSlayer& layer):
 fR(layer.fR),
@@ -1622,6 +1633,7 @@ fNMaxSigmaCl(layer.fNMaxSigmaCl)
 {
   //Copy constructor
 }
+*/
 //------------------------------------------------------------------------
 AliITStrackerMI::AliITSlayer::~AliITSlayer() {
   //--------------------------------------------------------------------
index 8eeee58..008b6ac 100644 (file)
@@ -172,7 +172,7 @@ fCluCoord(0)
   fVertexer = vertexer;
  
 }
-
+/*
 //____________________________________________________________________________
 AliITStrackerSA::AliITStrackerSA(const AliITStrackerSA& tracker):AliITStrackerMI(),
 fPhiEstimate(tracker.fPhiEstimate),
@@ -223,7 +223,7 @@ AliITStrackerSA& AliITStrackerSA::operator=(const AliITStrackerSA& source){
   return *this;
  
 }
-
+*/
 //____________________________________________________________________________
 AliITStrackerSA::~AliITStrackerSA(){
   // destructor
index 6c89ef3..69a9142 100644 (file)
@@ -36,8 +36,6 @@ class AliITStrackerSA : public AliITStrackerMI {
   AliITStrackerSA(const Char_t *geom);
   AliITStrackerSA(const Char_t *geom,AliESDVertex *vert);
   AliITStrackerSA(const Char_t *geom,AliITSVertexer *vertexer);
-  AliITStrackerSA(const AliITStrackerSA& tracker);
-  AliITStrackerSA& operator=(const AliITStrackerSA& source);
   virtual ~AliITStrackerSA();  
   virtual Int_t Clusters2Tracks(AliESDEvent *event);
   Int_t FindTracks(AliESDEvent* event, Bool_t useAllClusters=kFALSE);
@@ -128,6 +126,10 @@ class AliITStrackerSA : public AliITStrackerMI {
   TClonesArray** fCluLayer; //! array with clusters 
   TClonesArray** fCluCoord; //! array with cluster info
 
+ private:
+  AliITStrackerSA(const AliITStrackerSA& tracker);
+  AliITStrackerSA& operator=(const AliITStrackerSA& source);
+
   ClassDef(AliITStrackerSA,10)
 };
 
index 7777bc9..e8fffb5 100644 (file)
@@ -648,7 +648,7 @@ AliITStrackerV2::AliITSlayer::AliITSlayer():
   //--------------------------------------------------------------------
   
   for (Int_t i=0; i<kNsector; i++) fN[i]=0;
-
+  for (Int_t i=0; i<AliITSRecoParam::fgkMaxClusterPerLayer; i++)fIndex[i]=0;
 }
 
 AliITStrackerV2::AliITSlayer::
@@ -669,7 +669,7 @@ AliITSlayer(Double_t r,Double_t p,Double_t z,Int_t nl,Int_t nd):
   for (Int_t i=0; i<kNsector; i++) fN[i]=0;
 
   for (Int_t i=0; i<AliITSRecoParam::fgkMaxClusterPerLayer; i++) fClusters[i]=0;
-
+  for (Int_t i=0; i<AliITSRecoParam::fgkMaxClusterPerLayer; i++)fIndex[i]=0;
 }
 
 AliITStrackerV2::AliITSlayer::~AliITSlayer() {
@@ -1005,13 +1005,21 @@ Bool_t AliITStrackerV2::RefitAt(Double_t xx,AliITStrackV2 *t,
  
      {
      Double_t hI=i-0.5*step; 
-     if (TMath::Abs(hI-1.5)<0.01 || TMath::Abs(hI-3.5)<0.01) {             
-        Double_t rs=0.5*(fgLayers[i-step].GetR() + r);
-        Double_t d=0.0034, x0=38.6; 
-        if (TMath::Abs(hI-1.5)<0.01) {rs=9.; d=0.0097; x0=42;}
-        if (!t->PropagateTo(rs,-step*d,x0)) {
-          return kFALSE;
-        }
+     if (TMath::Abs(hI-1.5)<0.01 || TMath::Abs(hI-3.5)<0.01) {  
+       Int_t iLay = i-step;
+       Double_t rs = 0.;
+       if(iLay<0 || iLay>= AliITSgeomTGeo::kNLayers){
+        rs=0.5*(fgLayers[i-step].GetR() + r);
+       }
+       else{
+        AliError(Form("Invalid layer %d ",iLay));
+        return kFALSE;
+       }
+       Double_t d=0.0034, x0=38.6; 
+       if (TMath::Abs(hI-1.5)<0.01) {rs=9.; d=0.0097; x0=42;}
+       if (!t->PropagateTo(rs,-step*d,x0)) {
+        return kFALSE;
+       }
      }
      }