Coverity corrections.
authormorsch <morsch@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 23 Jan 2011 19:44:46 +0000 (19:44 +0000)
committermorsch <morsch@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 23 Jan 2011 19:44:46 +0000 (19:44 +0000)
STEER/AliAODCentrality.cxx
STEER/AliAODCluster.cxx
STEER/AliAODEvent.cxx
STEER/AliAODPid.cxx
STEER/AliAODRecoDecay.cxx
STEER/AliAODRedCov.h
STEER/AliAODTrack.cxx
STEER/AliMCVertex.h

index 160385c..1b1e88c 100644 (file)
@@ -117,6 +117,15 @@ fMultFMDA            (cnt.fMultFMDA ),
 fMultFMDC         (cnt.fMultFMDC )
 {
   // Copy constructor.
+  for (int i=0;i<6;i++) fNClusters[i] = cnt.fNClusters[i];
+  for (int i=0;i<2;i++) fNChips[i]    = cnt.fNChips[i];
+  for (int i=0;i<5;i++) fZNCtower[i]  = cnt.fZNCtower[i];
+  for (int i=0;i<5;i++) fZPCtower[i]  = cnt.fZPCtower[i];
+  for (int i=0;i<5;i++) fZNAtower[i]  = cnt.fZNAtower[i];
+  for (int i=0;i<5;i++) fZPAtower[i]  = cnt.fZPAtower[i];
+  for (int i=0;i<2;i++) fCentrZNC[i]  = cnt.fCentrZNC[i];
+  for (int i=0;i<2;i++) fCentrZNA[i]  = cnt.fCentrZNA[i];
+
 }
 
 //______________________________________________________________________________
index 1628556..e4d76e1 100644 (file)
@@ -60,6 +60,8 @@ AliAODCluster::AliAODCluster(Int_t id,
   fType(ttype)
 {
   // constructor
+  for (Int_t i = 0; i <  3; i++) fPosition[i] = 0.;
+  for (Int_t i = 0; i < 13; i++) fPID[i]      = 0;
  
   if(x)   {for (Int_t i = 0; i < 3  ; i++) SetPositionAt(x[i]  ,i);}
   if(pid) {for (Int_t i = 0; i < 13 ; i++) SetPIDAt     (pid[i],i);}
@@ -85,6 +87,9 @@ AliAODCluster::AliAODCluster(Int_t id,
   fType(ttype)
 {
   // constructor
+  for (Int_t i = 0; i <  3; i++) fPosition[i] = 0.;
+  for (Int_t i = 0; i < 13; i++) fPID[i]      = 0;
+
   if(x)   {for (Int_t i = 0; i < 3  ; i++) SetPositionAt(x[i]  ,i);}
   if(pid) {for (Int_t i = 0; i < 13 ; i++) SetPIDAt     (pid[i],i);}
   SetLabel(label, nLabel);
@@ -122,7 +127,7 @@ AliAODCluster::AliAODCluster(const AliAODCluster& clus) :
   // Copy constructor
 
   if(fPosition) {for(Int_t i = 0; i < 3  ; i++) fPosition[i] = clus.fPosition[i];}
-  if(fPID)      {for(Int_t i = 0; i < 13 ; i++) fPID[i]      = clus.fPID[i];}
+  for(Int_t i = 0; i < 13 ; i++) fPID[i]      = clus.fPID[i];
 
   SetLabel(clus.fLabel, clus.fNLabel);
 }
@@ -134,7 +139,7 @@ AliAODCluster& AliAODCluster::operator=(const AliAODCluster& clus)
   if(this!=&clus) {
     
        if(fPosition) {for(Int_t i = 0; i < 3 ;  i++) fPosition[i] = clus.fPosition[i];}
-       if(fPID)      {for(Int_t i = 0; i < 13 ; i++) fPID[i]      = clus.fPID[i];}
+       for(Int_t i = 0; i < 13 ; i++) fPID[i]      = clus.fPID[i];
     
     fEnergy = clus.fEnergy;
     fChi2 = clus.fChi2;
@@ -173,27 +178,22 @@ UShort_t AliAODCluster::GetMostProbablePID() const
   Int_t nPID = 13;
   UShort_t unknown = AliVCluster::kUnknown;
   
-  if (fPID) {
-    UShort_t loc = unknown;
-    Double_t max = 0.;
-    Bool_t allTheSame = kTRUE;
-    
-    for (Int_t iPID = 0; iPID < nPID; iPID++) {
-      if (fPID[iPID] >= max) {
-        if (fPID[iPID] > max) {
-          allTheSame = kFALSE;
-          max = fPID[iPID];
-          loc = (UShort_t)iPID;
-        } else {
-          allTheSame = kTRUE;
-        }
+  UShort_t loc = unknown;
+  Double_t max = 0.;
+  Bool_t allTheSame = kTRUE;
+  
+  for (Int_t iPID = 0; iPID < nPID; iPID++) {
+    if (fPID[iPID] >= max) {
+      if (fPID[iPID] > max) {
+       allTheSame = kFALSE;
+       max = fPID[iPID];
+       loc = (UShort_t)iPID;
+      } else {
+       allTheSame = kTRUE;
       }
     }
-    
-    return allTheSame ? unknown : loc;
-  } else {
-    return unknown;
   }
+  return allTheSame ? unknown : loc;
 }
 
 //______________________________________________________________________________
index 65ed50e..d52507d 100644 (file)
@@ -499,9 +499,8 @@ Int_t AliAODEvent::GetNumberOfMuonTracks() const
 {
   // get number of muon tracks
   Int_t nMuonTracks=0;
-  AliAODTrack *track = 0;
   for (Int_t iTrack = 0; iTrack < GetNTracks(); iTrack++) {
-    if ((track = GetTrack(iTrack))->IsMuonTrack()) {
+    if ((GetTrack(iTrack))->IsMuonTrack()) {
        nMuonTracks++;
     }
   }
index 41a5e2a..53195e1 100644 (file)
@@ -39,9 +39,11 @@ AliAODPid::AliAODPid():
     fHMPIDsignal(0)
 {
   // default constructor
-    for(Int_t i=0; i<kSPECIES; i++) fIntTime[i]=0; 
-    for(Int_t i=0; i<3; i++) fEMCALPosition[i] = 0.;
+    for(Int_t i=0; i<kSPECIES; i++) fIntTime[i]   = 0; 
+    for(Int_t i=0; i<3; i++) fEMCALPosition[i]    = 0.;
     for(Int_t i=0; i<5; i++) fTOFpidResolution[i] = 0.;
+    for(Int_t i=0; i<6; i++) fTRDmomentum[i]      = 0.;
+    for(Int_t i=0; i<3; i++) fEMCALMomentum[i]    = 0.;
   
 }
 
index d7d192b..6cbff57 100644 (file)
@@ -522,7 +522,7 @@ Int_t AliAODRecoDecay::MatchToMC(Int_t pdgabs,TClonesArray *mcArray,
     return -1;
   }
   
-  Int_t dgLabels[10];
+  Int_t dgLabels[10] = {0};
 
   // loop on daughters and write the labels
   for(Int_t i=0; i<ndg; i++) {
index cb4c620..edce3c3 100644 (file)
@@ -35,7 +35,10 @@ template <Int_t N> class AliAODRedCov {
    //
 
  public:
-  AliAODRedCov() {}
+  AliAODRedCov() {
+    for(Int_t i=0; i<N; i++)         fDiag[i]       = 0.;
+    for(Int_t i=0; i<N*(N-1)/2; i++) fODia[i]       = 0.;
+  }
   virtual ~AliAODRedCov() {}
   template <class T> void GetCovMatrix(T *cmat) const;
   template <class T> void SetCovMatrix(T *cmat);
index fa384d8..176e052 100644 (file)
@@ -346,27 +346,22 @@ AliAODTrack::AODTrkPID_t AliAODTrack::GetMostProbablePID() const
   // Returns the most probable PID array element.
   
   Int_t nPID = 10;
-  if (fPID) {
-    AODTrkPID_t loc = kUnknown;
-    Double_t max = 0.;
-    Bool_t allTheSame = kTRUE;
-    
-    for (Int_t iPID = 0; iPID < nPID; iPID++) {
-      if (fPID[iPID] >= max) {
-       if (fPID[iPID] > max) {
-         allTheSame = kFALSE;
-         max = fPID[iPID];
-         loc = (AODTrkPID_t)iPID;
-       } else {
-         allTheSame = kTRUE;
-       }
+  AODTrkPID_t loc = kUnknown;
+  Double_t max = 0.;
+  Bool_t allTheSame = kTRUE;
+  
+  for (Int_t iPID = 0; iPID < nPID; iPID++) {
+    if (fPID[iPID] >= max) {
+      if (fPID[iPID] > max) {
+       allTheSame = kFALSE;
+       max = fPID[iPID];
+       loc = (AODTrkPID_t)iPID;
+      } else {
+       allTheSame = kTRUE;
       }
     }
-    
-    return allTheSame ? kUnknown : loc;
-  } else {
-    return kUnknown;
   }
+  return allTheSame ? kUnknown : loc;
 }
 
 //______________________________________________________________________________
index 1d1aa7b..fb5f2ec 100644 (file)
@@ -15,7 +15,7 @@
 
 class AliMCVertex : public AliVVertex {
     public:
-    AliMCVertex() {;}
+  AliMCVertex() {fPosition[0] = 0.; fPosition[1] = 0.; fPosition[2] = 0.;}
     AliMCVertex(Double_t xv, Double_t yv, Double_t zv)
        {fPosition[0] = xv; fPosition[1] = yv; fPosition[2] = zv;}
     virtual ~AliMCVertex() {;}