coverity fix
authorjotwinow <jotwinow@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 31 Mar 2011 18:07:52 +0000 (18:07 +0000)
committerjotwinow <jotwinow@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 31 Mar 2011 18:07:52 +0000 (18:07 +0000)
PWG1/TPC/AliComparisonDraw.cxx
PWG1/TPC/AliESDRecInfo.cxx
PWG1/TPC/AliESDRecKinkInfo.cxx
PWG1/TPC/AliESDRecV0Info.cxx
PWG1/TPC/AliGenInfoMaker.cxx
PWG1/TPC/AliGenInfoTask.cxx
PWG1/TPC/AliGenKinkInfo.cxx
PWG1/TPC/AliGenV0Info.cxx

index 002c967..3cd88fa 100644 (file)
@@ -490,7 +490,7 @@ TGraph2D * AliComparisonDraw::MakeStat2D(TH3 * his, Int_t delta0, Int_t delta1,
     for (Int_t iy=0; iy<nbiny;iy++){
       Float_t xcenter = xaxis->GetBinCenter(ix); 
       Float_t ycenter = yaxis->GetBinCenter(iy); 
-      sprintf(name,"%s_%d_%d",his->GetName(), ix,iy);
+      snprintf(name,1000,"%s_%d_%d",his->GetName(), ix,iy);
       TH1 *projection = his->ProjectionZ(name,ix-delta0,ix+delta0,iy-delta1,iy+delta1);
       Float_t stat= 0;
       if (type==0) stat = projection->GetMean();
@@ -532,7 +532,7 @@ TGraph * AliComparisonDraw::MakeStat1D(TH3 * his, Int_t delta1, Int_t type){
   for (Int_t ix=0; ix<nbinx;ix++){
     Float_t xcenter = xaxis->GetBinCenter(ix); 
     //    Float_t ycenter = yaxis->GetBinCenter(iy); 
-    sprintf(name,"%s_%d",his->GetName(), ix);
+    snprintf(name,1000,"%s_%d",his->GetName(), ix);
     TH1 *projection = his->ProjectionZ(name,ix-delta1,ix+delta1,0,nbiny);
     Float_t stat= 0;
     if (type==0) stat = projection->GetMean();
index 71c3829..24bf4b1 100644 (file)
@@ -81,6 +81,31 @@ AliESDRecInfo::AliESDRecInfo():
   //
   //  default constructor
   //
+  for(Int_t i=0; i<10; i++) { fTPCPoints[i] = 0;}
+  for(Int_t i=0; i<5; i++) {
+    fTPCinR0[i] = 0 ;
+    fTPCinR1[i] = 0 ;
+    fTPCinP0[i] = 0 ;
+    fTPCinP1[i] = 0 ;
+    fTPCDelta[i] = 0 ;
+    fTPCPools[i] = 0 ;
+    fITSinR0[i] = 0 ;
+    fITSinR1[i] = 0 ;
+    fITSinP0[i] = 0 ;
+    fITSinP1[i] = 0 ;
+    fITSDelta[i] = 0 ;
+    fITSPools[i] = 0 ;
+  }
+  for(Int_t i=0; i<2; i++) {
+    fTPCAngle0[i] =  0 ;
+    fTPCAngle1[i] =  0 ;
+    fITSAngle0[i] =  0 ;
+    fITSAngle1[i] =  0 ;
+    fLabels[i] =  0 ;
+  }
+  for(Int_t i=0; i<3; i++) { fTRLocalCoord[i] =  0 ; }
+  for(Int_t i=0; i<4; i++) { fStatus[i] =  0 ; }
+
 }
 
 
@@ -105,10 +130,36 @@ AliESDRecInfo::AliESDRecInfo(const AliESDRecInfo& recinfo):
   //
   //
   //
+  for(Int_t i=0; i<10; i++) { fTPCPoints[i] = 0;}
+  for(Int_t i=0; i<5; i++) {
+    fTPCinR0[i] = 0 ;
+    fTPCinR1[i] = 0 ;
+    fTPCinP0[i] = 0 ;
+    fTPCinP1[i] = 0 ;
+    fTPCDelta[i] = 0 ;
+    fTPCPools[i] = 0 ;
+    fITSinR0[i] = 0 ;
+    fITSinR1[i] = 0 ;
+    fITSinP0[i] = 0 ;
+    fITSinP1[i] = 0 ;
+    fITSDelta[i] = 0 ;
+    fITSPools[i] = 0 ;
+  }
+  for(Int_t i=0; i<2; i++) {
+    fTPCAngle0[i] =  0 ;
+    fTPCAngle1[i] =  0 ;
+    fITSAngle0[i] =  0 ;
+    fITSAngle1[i] =  0 ;
+    fLabels[i] =  0 ;
+  }
+  for(Int_t i=0; i<3; i++) { fTRLocalCoord[i] =  0 ; }
+  for(Int_t i=0; i<4; i++) { fStatus[i] =  0 ; }
+
   memcpy(this,&recinfo, sizeof(recinfo));
   fESDtrack=0; fTrackF=0; fTPCtrack=0;fITStrack=0;fTRDtrack=0;
   fTracks=0;
   SetESDtrack(recinfo.GetESDtrack());
+
 }
 
 
@@ -172,14 +223,16 @@ void AliESDRecInfo::SetESDtrack(const AliESDtrack *track){
     Int_t icalib=0;
     TObject *cobject=0;
     //
+    AliTPCseed *tpcSeed = NULL;
+    AliTRDtrackV1 *trdTrack = NULL;
     while (fTrackF->GetCalibObject(icalib)){
       cobject=fTrackF->GetCalibObject(icalib);
-      if (dynamic_cast<AliTPCseed*>(cobject)){
+      if ((tpcSeed = dynamic_cast<AliTPCseed*>(cobject))){
        if (fTPCtrack) delete fTPCtrack;
-       fTPCtrack = (AliTPCseed*)(dynamic_cast<AliTPCseed*>(cobject))->Clone();
-      } else if (dynamic_cast<AliTRDtrackV1*>(cobject)){
+       fTPCtrack = (AliTPCseed*)tpcSeed->Clone();
+      } else if ((trdTrack = dynamic_cast<AliTRDtrackV1*>(cobject))){
        if (fTRDtrack) delete fTRDtrack;
-       fTRDtrack = (AliTRDtrackV1*)(dynamic_cast<AliTRDtrackV1*>(cobject))->Clone();
+       fTRDtrack = (AliTRDtrackV1*)trdTrack->Clone();
       }
       icalib++;
     }
index 9d7c681..7c12437 100644 (file)
@@ -84,6 +84,13 @@ AliESDRecKinkInfo::AliESDRecKinkInfo():
   //
   // Default constructor
   //
+  for (Int_t i=0; i<3; i++) {
+    fPdr[i] = 0;
+    fXr[i] = 0;
+    fPm[i] = 0;
+    fAngle[i] = 0;
+  }
+  for (Int_t i=0; i<2; i++) { fLab[i] = 0; } 
 }
 
 ////
@@ -117,7 +124,8 @@ void  AliESDRecKinkInfo::Update()
     //
     //find intersection linear
     //
-    Double_t phase[2][2],radius[2];
+    Double_t phase[2][2] = { {0,0}, {0,0}};
+    Double_t radius[2] = {0} ;
     Int_t  points = dhelix1.GetRPHIintersections(mhelix, phase, radius,200);
     Double_t delta1=10000,delta2=10000;  
 
index 9885a84..74d4fea 100644 (file)
@@ -104,6 +104,17 @@ AliESDRecV0Info:: AliESDRecV0Info():
   //
   // default constructor
   //
+  for (Int_t i=0; i<3; i++) {
+    fPdr[i] = 0;
+    fXr[i] = 0;
+    fPm[i] = 0;
+    fAngle[i] = 0;
+  }
+  for (Int_t i=0; i<2; i++) {
+    fRs[i] = 0;
+    fLab[i] = 0;
+  }
+
   fV0tpc    = new AliV0();
   fV0its    = new AliV0();
   fV0rec    = new AliV0();
@@ -154,7 +165,8 @@ void  AliESDRecV0Info::Update(Float_t vertex[3])
     //
     //find intersection linear
     //
-    Double_t phase[2][2],radius[2];
+    Double_t phase[2][2] = { {0,0},{0,0} }; 
+    Double_t radius[2] = {0};
     Int_t  points = dhelix1.GetRPHIintersections(mhelix, phase, radius,200);
     Double_t delta1=10000,delta2=10000;  
 
@@ -212,7 +224,8 @@ void  AliESDRecV0Info::Update(Float_t vertex[3])
     
     distance2 = TMath::Min(delta1,delta2);
     if (distance2>100) fDist2 =100;
-    return;
+    Bool_t checkAll = 0; // to be checked
+    if(checkAll) { 
     if (delta1<delta2){
       //get V0 info
       dhelix1.Evaluate(phase[0][0],fXr);
@@ -259,8 +272,10 @@ void  AliESDRecV0Info::Update(Float_t vertex[3])
     fPointAngleTh = (v[2]*p[2]+vnorm2*pnorm2)/(vnorm3*pnorm3);  
     fPointAngle   = (v[0]*p[0]+v[1]*p[1]+v[2]*p[2])/(vnorm3*pnorm3);
   }
+  }
 }
 
+
 void  AliESDRecV0Info::Reset(){
   //
   // Reset status and all counters
index 1ba1572..0da3d15 100644 (file)
@@ -101,7 +101,8 @@ AliGenInfoMaker::AliGenInfoMaker():
   fTRDPtCut(0.1),              //  TRD pt cut
   fTOFPtCut(0.1)               //  TOF pt cut
 {    
-  sprintf(fFnRes,"%s","genTracks.root");
+  snprintf(fFnRes,1000,"%s","genTracks.root");
+  for(Int_t i=0; i<3; i++) { fVPrim[i] = 0; };  
 }
 
 
@@ -138,10 +139,11 @@ AliGenInfoMaker::AliGenInfoMaker(const char * fnGalice, const char* fnRes,
   //
   // 
   //
+  for(Int_t i=0; i<3; i++) { fVPrim[i] = 0; };  
   fFirstEventNr = firstEvent;
   fEventNr = firstEvent;
   fNEvents = nEvents;
-  sprintf(fFnRes,"%s",fnRes);
+  snprintf(fFnRes,1000,"%s",fnRes);
   //
   //
   //
@@ -431,7 +433,7 @@ Int_t AliGenInfoMaker::TreeKLoop()
        <<fEventNr<<endl;
   }  
   Int_t  ipdg = 0;
-  TParticlePDG *ppdg = 0;
+  //TParticlePDG *ppdg = 0;
   // not all generators give primary vertex position. Take the vertex
   // of the particle 0 as primary vertex.
   TDatabasePDG  pdg; //get pdg table  
@@ -458,7 +460,7 @@ Int_t AliGenInfoMaker::TreeKLoop()
     //
     ipdg = info->fParticle.GetPdgCode();
     info->fPdg = ipdg;
-    ppdg = pdg.GetParticle(ipdg);         
+    //ppdg = pdg.GetParticle(ipdg);       
     info->fEventNr = fEventNr;
     info->Update();
     if (fGenTracksArray){
index 72465dd..f63d51b 100644 (file)
@@ -327,6 +327,7 @@ void AliGenInfoTask::ProcessESDInfo(){
   //
   //
   //
+  if (!fESD) return;
   static AliTPCParamSR param;
   fESDfriend=static_cast<AliESDfriend*>(fESD->FindListObject("AliESDfriend"));
   if (!fESDfriend) {
@@ -336,7 +337,6 @@ void AliGenInfoTask::ProcessESDInfo(){
   fESD->SetESDfriend(fESDfriend);
   //
   //
-  if (!fESD) return;
   Int_t ntracks = fESD->GetNumberOfTracks();
   for (Int_t itrack=0; itrack<ntracks; itrack++){
     AliESDtrack *track = fESD->GetTrack(itrack);
index 5164c19..7049eec 100644 (file)
@@ -84,6 +84,7 @@ AliGenKinkInfo::AliGenKinkInfo():
     fMCX[i]=0;
     fMCPm[i]=0;
     fMCAngle[i]=0;
+    fMCXr[i] = 0;
   }
   for (Int_t i=0; i<2; i++) {
     fPdg[i]= 0;
@@ -154,7 +155,8 @@ void AliGenKinkInfo::Update()
   //find intersection linear
   //
   Double_t distance1, distance2;
-  Double_t phase[2][2],radius[2];
+  Double_t phase[2][2] = { {0,0},{0,0} };
+  Double_t radius[2] = {0};
   Int_t  points = dhelix1.GetRPHIintersections(mhelix, phase, radius);
   Double_t delta1=10000,delta2=10000;    
   if (points>0){
index 45cc8e7..2af08f9 100644 (file)
@@ -156,7 +156,8 @@ void AliGenV0Info::Update(Float_t vertex[3])
   //find intersection linear
   //
   Double_t distance1, distance2;
-  Double_t phase[2][2],radius[2];
+  Double_t phase[2][2] = {{0,0},{0,0}};
+  Double_t radius[2] = {0};
   Int_t  points = dhelix1.GetRPHIintersections(mhelix, phase, radius);
   Double_t delta1=10000,delta2=10000;    
   if (points>0){