Coverity corrections
authormorsch <morsch@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 23 Jan 2011 19:34:42 +0000 (19:34 +0000)
committermorsch <morsch@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 23 Jan 2011 19:34:42 +0000 (19:34 +0000)
JETAN/AliAnalysisTaskKMeans.cxx
JETAN/AliEventShape.cxx
JETAN/AliFastJetInput.cxx
JETAN/AliJetBkg.h
JETAN/AliJetHadronCorrectionv1.cxx
JETAN/AliJetHadronCorrectionv1.h
JETAN/AliJetUnitArray.cxx
JETAN/AliJetUnitArray.h
JETAN/AliSISConeJetFinder.cxx
JETAN/AliUA1JetFinderV2.cxx

index 19067c5..819ec8b 100644 (file)
@@ -492,7 +492,6 @@ void AliAnalysisTaskKMeans::UserExec(Option_t *)
   
     mPhi    = best.GetMx();
     mEta    = best.GetMy();
-    sigma2  = best.GetSigma2();
     nk      = best.GetK();
     im      = (best.GetInd())[0];
     etaC    = mEta[im];    
index a91e246..5864a47 100644 (file)
@@ -108,7 +108,12 @@ TArrayD * AliEventShape::GetThrustParamMC(AliMCEvent* mcEvent, Int_t  nstudymin,
   pxT = new Double_t[nmctracks];
   pyT = new Double_t[nmctracks];
   ptT = new Double_t[nmctracks];
-
+  for (Int_t i = 0; i < nmctracks; i++)
+    {
+      pxT[i] = 0;
+      pyT[i] = 0;
+      ptT[i] = 0;
+    }
   for (Int_t iMCTracks = 0; iMCTracks < nPrim; ++iMCTracks) {    
       TParticle* trackmc = stack->Particle(iMCTracks);
       if (!trackmc) continue;
index e9a5090..2680165 100644 (file)
@@ -95,9 +95,7 @@ void AliFastJetInput::FillInput(){
   Int_t fOpt   = fReader->GetReaderHeader()->GetDetector();
 
   // check if we are reading AOD jets
-  TRefArray *refs = 0;
   Bool_t fromAod = !strcmp(fReader->ClassName(),"AliJetAODReader");
-  if (fromAod) { refs = fReader->GetReferences(); }
   
   // RUN ALGORITHM  
   // read input particles -----------------------------
index 38a5299..b901e37 100644 (file)
@@ -37,7 +37,7 @@ class AliJetBkg : public TObject
     Float_t BkgChargedFastJet();
     Float_t BkgStat();
     Float_t BkgFastJetCone(TClonesArray* fAODJets);
-    Float_t BkgRemoveJetLeading(TClonesArray* fAODJets);
+//    Float_t BkgRemoveJetLeading(TClonesArray* fAODJets);
     Float_t BkgRemoveJetLeadingFromUArray(TClonesArray* fAODJets);
     Float_t EtaToTheta(Float_t arg);
     Bool_t EmcalAcceptance(const Float_t eta, const Float_t phi, const Float_t radius);
index fa04335..799517f 100644 (file)
@@ -52,6 +52,7 @@ AliJetHadronCorrectionv1::AliJetHadronCorrectionv1(const char *name,const char *
      fSamplingFraction(0)
 {
   fgHadrCorr = this;
+  for (Int_t i = 0; i < 8; i++) fPar[i] = 0.;
 }
 
 AliJetHadronCorrectionv1*
index 69daa8f..643425a 100644 (file)
@@ -27,7 +27,7 @@ class AliJetHadronCorrectionv1 : public AliJetHadronCorrection
   void TrackPositionEMCal(AliAODTrack* track,Double_t &eta, Double_t &phi);
 
  protected:
-  AliJetHadronCorrectionv1():fSamplingFraction(0) {;}
+  AliJetHadronCorrectionv1():fSamplingFraction(0) {for (Int_t i = 0; i < 8; i++) fPar[i] = 0.;}
   AliJetHadronCorrectionv1(const char *name, const char *title);
 
  private:
index 4daa190..77d28f3 100644 (file)
@@ -254,30 +254,30 @@ Float_t  AliJetUnitArray::EtaToTheta(Float_t arg) const
 }
 
 //------------------------------------------------------------------------
-Bool_t AliJetUnitArray::operator>(AliJetUnitArray unit) const
+Bool_t AliJetUnitArray::operator>(AliJetUnitArray* unit) const
 {
   // Greater than operator used by sort
-  if( fUnitEnergy > unit.GetUnitEnergy())
+  if( fUnitEnergy > unit->GetUnitEnergy())
     return kTRUE;
   else 
     return kFALSE;
 }
 
 //------------------------------------------------------------------------
-Bool_t AliJetUnitArray::operator<( AliJetUnitArray unit) const
+Bool_t AliJetUnitArray::operator<( AliJetUnitArray* unit) const
 {
   // Less than operator used by sort
-  if( fUnitEnergy < unit.GetUnitEnergy())
+  if( fUnitEnergy < unit->GetUnitEnergy())
     return kTRUE;
   else
     return kFALSE;
 }
 
 //------------------------------------------------------------------------
-Bool_t AliJetUnitArray::operator==( AliJetUnitArray unit) const
+Bool_t AliJetUnitArray::operator==( AliJetUnitArray* unit) const
 {
   // equality operator used by sort
-  if( fUnitEnergy == unit.GetUnitEnergy())
+  if( fUnitEnergy == unit->GetUnitEnergy())
     return kTRUE;
   else
     return kFALSE;
index 5300bc2..804e377 100644 (file)
@@ -131,9 +131,9 @@ class AliJetUnitArray : public TObject
  protected:
   AliJetUnitArray(const AliJetUnitArray& rUnit);
   AliJetUnitArray& operator = (const AliJetUnitArray& rhs);
-  Bool_t operator>  ( AliJetUnitArray unit1) const;
-  Bool_t operator<  ( AliJetUnitArray unit1) const;
-  Bool_t operator== ( AliJetUnitArray unit1) const;
+  Bool_t operator>  ( AliJetUnitArray* unit1) const;
+  Bool_t operator<  ( AliJetUnitArray* unit1) const;
+  Bool_t operator== ( AliJetUnitArray* unit1) const;
 
   Float_t                            fUnitEnergy;       // Energy (Pt,et) of the unit 
   Float_t                            fUnitEta;          // Eta of the unit
index 5462898..decadb9 100644 (file)
@@ -89,10 +89,7 @@ void AliSISConeJetFinder::FindJets()
   Int_t fOpt    = fReader->GetReaderHeader()->GetDetector();
 
   // Check if we are reading AOD jets
-  TRefArray *refs = 0;
   Bool_t fromAod = !strcmp(fReader->ClassName(),"AliJetAODReader");
-  if (fromAod) { refs = fReader->GetReferences(); }
-  
 
 //******************************** SISCONE PLUGIN CONFIGURATION
 // Here we look for SISCone parameters in the header and we define our plugin.  
index 76a4743..3098024 100644 (file)
@@ -97,7 +97,15 @@ void AliUA1JetFinderV2::FindJetsC()
   Int_t*   cFlagT = new Int_t[nIn]; // Temporarily added
   Int_t*   sFlagT = new Int_t[nIn]; // Temporarily added
   Int_t*   injet  = new Int_t[nIn];
-  
+
+  for (Int_t i = 0; i < nIn; i++) {
+    ptT[i]    = 0.;
+    etaT[i]   = 0.;
+    phiT[i]   = 0.;
+    cFlagT[i] = 0; 
+    sFlagT[i] = 0; 
+    injet[i]  = 0;
+  }
   //total energy in array
   Float_t  etbgTotal = 0.0;
   TH1F*    hPtTotal  = new TH1F("hPt","Pt distribution of all particles ",100,0.0,15.0);
@@ -342,7 +350,16 @@ void AliUA1JetFinderV2::FindJets()
   Float_t *etaCell2  = new Float_t[nIn]; //! Cell eta - Extracted from UnitArray
   Float_t *phiCell2  = new Float_t[nIn]; //! Cell phi - Extracted from UnitArray
   Int_t   *flagCell2 = new Int_t[nIn];  //! Cell phi - Extracted from UnitArray
-
+  for(Int_t i = 0; i < nIn; i++) {
+    etCell[i]    = 0.;
+    etaCell[i]   = 0.;
+    phiCell[i]   = 0.;
+    flagCell[i]  = 0.;
+    etCell2[i]   = 0.;
+    etaCell2[i]  = 0.;
+    phiCell2[i]  = 0.;
+    flagCell2[i] = 0.;
+  }
   // Information extracted from fUnitArray
   // Load input vectors and calculate total energy in array
   for(Int_t i=0; i<nIn; i++) 
@@ -743,10 +760,10 @@ void AliUA1JetFinderV2::RunAlgoritm(Int_t nIn, Float_t* etCell, Float_t* const e
   Float_t etseed  = header->GetEtSeed();
 
   // Tmp array of jets form algoritm
-  Float_t etaAlgoJet[30];
-  Float_t phiAlgoJet[30];
-  Float_t etAlgoJet[30];
-  Int_t   ncellsAlgoJet[30];
+  Float_t etaAlgoJet[30]    = {0.};
+  Float_t phiAlgoJet[30]    = {0.};
+  Float_t etAlgoJet[30]     = {0.};
+  Int_t   ncellsAlgoJet[30] = {0.};
 
   // Run algorithm//
 
@@ -1129,11 +1146,9 @@ void AliUA1JetFinderV2::SubtractBackg(const Int_t& nIn, const Int_t&nJ, Float_t&
   AliUA1JetHeaderV1* header = (AliUA1JetHeaderV1*) fHeader;
   fOpt = fReader->GetReaderHeader()->GetDetector();
   Float_t rc= header->GetRadius();
-  Float_t etIn[30];
+  Float_t etIn[30] = {0.};
   Float_t etOut = 0;
-  
-  for(Int_t j=0;j<30;j++){etIn[j]=0.;}
-  
+    
   for(Int_t jpart = 0; jpart < nIn; jpart++){ // loop for all particles in array
 
     for(Int_t ijet=0; ijet<nJ; ijet++){
@@ -1398,7 +1413,7 @@ void AliUA1JetFinderV2::SubtractBackgStat(const Int_t& nIn, const Int_t&nJ,Float
   
   //calculate energy inside
   Float_t rc= header->GetRadius();
-  Float_t etIn[30];
+  Float_t etIn[30] = {0.};
   
   for(Int_t jpart = 0; jpart < nIn; jpart++)
     { // loop for all particles in array