various coverity fixes

diff --git a/PWGCF/Correlations/DPhi/AliAnalysisTaskDptDptCorrelations.cxx b/PWGCF/Correlations/DPhi/AliAnalysisTaskDptDptCorrelations.cxx
index d7d58eff4429f918bc6ed953dbf99a7630b417e3..7b6170add6880fd01dde4274848e0d00477024d1 100644 (file)
--- a/PWGCF/Correlations/DPhi/AliAnalysisTaskDptDptCorrelations.cxx
+++ b/PWGCF/Correlations/DPhi/AliAnalysisTaskDptDptCorrelations.cxx
@@ -394,6 +394,8 @@ _dedxMax              ( 100000),
 _nClusterMin          ( 80), 
 _trackFilterBit       ( 0),
 fNSigmaCut            ( 3.),
+_tpcnclus             ( 50),
+_chi2ndf              (5.),
 _field    ( 1.),
 _nTracks  ( 0 ),
 _mult0    ( 0 ),

diff --git a/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskAODFilterBitQA.cxx b/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskAODFilterBitQA.cxx
index cb080bd98050f9219baa8e230537ca0606ca8e72..4471bd36ec0b104e78febc3f4d23ad5e126ddcdc 100755 (executable)
--- a/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskAODFilterBitQA.cxx
+++ b/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskAODFilterBitQA.cxx
@@ -261,18 +261,19 @@ void AliAnalysisTaskAODFilterBitQA::GetAcceptedTracks(AliVEvent *event, Double_t
 
 
     // AOD track cuts
-    for(Int_t iTrackBit = 0; iTrackBit < gBitMax; iTrackBit++){
-      fHistTrackStats->Fill(gCentrality,iTrackBit,aodTrack->TestFilterBit(1<<iTrackBit));
-      
-      if(aodTrack->TestFilterBit(1<<iTrackBit)){
-       fHistKinematics[iCharge][iTrackBit]->Fill(vEta,vPhi,vPt);
-       fHistDCAconstrained[iCharge][iTrackBit]->Fill(vDCAconstrainedxy,vDCAconstrainedz);
-       fHistDCAglobal[iCharge][iTrackBit]->Fill(vDCAglobalx,vDCAglobaly,vDCAglobalz);
-       fHistChiClus[iCharge][iTrackBit]->Fill(vChi2,vClus);
-      }
-      
-    }//bit loop
+    if(iCharge > -1){
+      for(Int_t iTrackBit = 0; iTrackBit < gBitMax; iTrackBit++){
+       fHistTrackStats->Fill(gCentrality,iTrackBit,aodTrack->TestFilterBit(1<<iTrackBit));
+       
+       if(aodTrack->TestFilterBit(1<<iTrackBit)){
+         fHistKinematics[iCharge][iTrackBit]->Fill(vEta,vPhi,vPt);
+         fHistDCAconstrained[iCharge][iTrackBit]->Fill(vDCAconstrainedxy,vDCAconstrainedz);
+         fHistDCAglobal[iCharge][iTrackBit]->Fill(vDCAglobalx,vDCAglobaly,vDCAglobalz);
+         fHistChiClus[iCharge][iTrackBit]->Fill(vChi2,vClus);
+       } 
+      }//bit loop
+    }//charge positive or negative
   }//track loop
-
+  
   return;  
 }

diff --git a/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskEffContBF.cxx b/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskEffContBF.cxx
index 6d88ab9d3676a44ea568bc933e8e1ea9f66ebf41..37458f195ef087af6f541fb8fe6131bd08b5e0df 100644 (file)
--- a/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskEffContBF.cxx
+++ b/PWGCF/EBYE/BalanceFunctions/AliAnalysisTaskEffContBF.cxx
@@ -348,8 +348,10 @@ void AliAnalysisTaskEffContBF::UserExec(Option_t *) {
     
     // AOD only
     AliAODMCHeader* header = (AliAODMCHeader*) fAOD->GetList()->FindObject(AliAODMCHeader::StdBranchName());
-    if (!header)
+    if (!header){
       AliFatal("fInjectedSignals set but no MC header found");
+      return;
+    }
     
     headers = header->GetNCocktailHeaders();
     eventHeader = header->GetCocktailHeader(0);

diff --git a/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.cxx b/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.cxx
index 77ce26e3b21d7514faacf19574b1ff2e44fb584f..17ed978e1ced82efbe3a74f5e890f22a2e366565 100644 (file)
--- a/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.cxx
+++ b/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.cxx
@@ -34,7 +34,7 @@ ClassImp(AliEbyEMultFluctuationTask)
 
 //________________________________________________________________________
 AliEbyEMultFluctuationTask::AliEbyEMultFluctuationTask(const char *name) 
-: AliAnalysisTaskSE(name),fAOD(0), fAODVertex(0),fHistNchPt(0),fHistNchEta(0),fHistNchEtaCent(0),fHistNchPhi(0),fHistDCAxy(0),fHistDCAz(0),fHistnclus(0),fHistchi2ndf(0),fHistchi2ndfvscs(0),fHistVz(0),fHistMultV0A(0),fHistMultV0C(0),fHistMultV0total(0),My_ntuple(0),fOutputList(0),fCentralityEstimator("V0M"),fCentralityCounter(0),fEventCounter(0),histcounter(0)
+: AliAnalysisTaskSE(name),fAOD(0), fAODVertex(0),fHistNchPt(0),fHistNchEta(0),fHistNchEtaCent(0),fHistNchPhi(0),fHistDCAxy(0),fHistDCAz(0),fHistnclus(0),fHistchi2ndf(0),fHistchi2ndfvscs(0),fHistVz(0),fHistMultV0A(0),fHistMultV0C(0),fHistMultV0total(0),My_ntuple(0),fOutputList(0),fCentralityEstimator("V0M"),fCentralityBins20(kFALSE),fCentralityCounter(0),fEventCounter(0),histcounter(0)
 
 
 {
@@ -61,11 +61,12 @@ AliEbyEMultFluctuationTask::AliEbyEMultFluctuationTask(const char *name)
 //________________________________________________________________________
 Bool_t AliEbyEMultFluctuationTask :: SelectEvent(AliAODVertex* vertex)
 {
-if(vertex) 
-  
-      if(vertex->GetNContributors() < 0) return kFALSE;
 
-         
+  if(vertex){ 
+  
+    if(vertex->GetNContributors() < 0) return kFALSE;
+    
+    
          Double_t lvx = vertex->GetX();
          Double_t lvy = vertex->GetY();
          Double_t lvz = vertex->GetZ();
@@ -74,13 +75,17 @@ if(vertex)
          
          
          if(TMath::Abs(lvx) > 0.3)  return kFALSE;
-           if(TMath::Abs(lvy) > 0.3) return kFALSE;
-           if(TMath::Abs(lvz) > 10) return kFALSE;
-           if(vertex->GetType()==AliAODVertex::kPileupSPD) return kFALSE;  
-            
-               fEventCounter->Fill(5);
-               fHistVz->Fill(lvz);             
-return kTRUE;
+         if(TMath::Abs(lvy) > 0.3) return kFALSE;
+         if(TMath::Abs(lvz) > 10) return kFALSE;
+         if(vertex->GetType()==AliAODVertex::kPileupSPD) return kFALSE;  
+         
+         fEventCounter->Fill(5);
+         fHistVz->Fill(lvz);           
+         return kTRUE;
+  }
+  else{
+    return kFALSE;
+  }
 }
 //_____________________________________________________________________________
 Int_t AliEbyEMultFluctuationTask :: SelectTrack(AliAODTrack* track)
@@ -291,13 +296,12 @@ if(!isSelected) return;
 
 // Get the event vertex.
        fAODVertex = fAOD->GetPrimaryVertex();
-       
-       if(TMath::Abs((fAODVertex->GetZ())-((fAOD->GetPrimaryVertexSPD())->GetZ())) > 0.5) return;
 
-       if (!fAODVertex) {
-               
-               return;
+       if (!fAODVertex) {      
+         return;
        }
+       
+       if(TMath::Abs((fAODVertex->GetZ())-((fAOD->GetPrimaryVertexSPD())->GetZ())) > 0.5) return;
 
 
     if (!SelectEvent(fAODVertex)) return;

diff --git a/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.h b/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.h
index f2a0d7bef04132c86bd6bc93e7d1db389c80c856..2867a43af44dda1a67e7e47e9b7da45062fc945a 100644 (file)
--- a/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.h
+++ b/PWGCF/EBYE/Fluctuations/AliEbyEMultFluctuationTask.h
@@ -18,7 +18,7 @@ class AliAODTracklets;
 
 class AliEbyEMultFluctuationTask : public AliAnalysisTaskSE {
  public:
- AliEbyEMultFluctuationTask() : AliAnalysisTaskSE(), fAOD(0), fAODVertex(0),fHistNchPt(0),fHistNchEta(0),fHistNchEtaCent(0),fHistNchPhi(0),fHistDCAxy(0),fHistDCAz(0),fHistnclus(0),fHistchi2ndf(0),fHistchi2ndfvscs(0),fHistVz(0),fHistMultV0A(0),fHistMultV0C(0),fHistMultV0total(0),My_ntuple(0),fOutputList(0),fCentralityEstimator("V0M"),fCentralityCounter(0),fEventCounter(0),histcounter(0){
+ AliEbyEMultFluctuationTask() : AliAnalysisTaskSE(), fAOD(0), fAODVertex(0),fHistNchPt(0),fHistNchEta(0),fHistNchEtaCent(0),fHistNchPhi(0),fHistDCAxy(0),fHistDCAz(0),fHistnclus(0),fHistchi2ndf(0),fHistchi2ndfvscs(0),fHistVz(0),fHistMultV0A(0),fHistMultV0C(0),fHistMultV0total(0),My_ntuple(0),fOutputList(0),fCentralityEstimator("V0M"),fCentralityBins20(kFALSE),fCentralityCounter(0),fEventCounter(0),histcounter(0){
                for(Int_t ibin=0;ibin<91;ibin++)
                {
                        fMult[ibin]=NULL;