Coverity fixes
authormvl <marco.van.leeuwen@cern.ch>
Mon, 12 Jan 2015 16:07:03 +0000 (17:07 +0100)
committermvl <marco.van.leeuwen@cern.ch>
Tue, 13 Jan 2015 09:10:37 +0000 (10:10 +0100)
PWGJE/EMCALJetTasks/UserTasks/AliAnalysisTaskJetJTJT.cxx
PWGJE/FlavourJetTasks/AliAnalysisTaskEmcalJetV0CF.cxx
PWGJE/FlavourJetTasks/AliAnalysisTaskEmcalJetV0Filter.cxx
PWGJE/PWGJE/AliAnalysisTaskJetCore.cxx

index a690e74..62656e9 100644 (file)
@@ -1086,7 +1086,7 @@ bool JTJTEfficiency::Load(){
        fCentBinAxis = (TAxis*)fEffDir[2]->Get("CentralityBin");
        if( !fCentBinAxis ){
                cout << "J_ERROR : No CentralityBin in directory" << endl;
-               gSystem->Exit(1);
+                return false;
        }
 
 
index 8294b02..c54d238 100644 (file)
@@ -298,6 +298,8 @@ Bool_t AliAnalysisTaskEmcalJetV0CF::FillRecoInfo()
   Double_t dZNA = fCentInfo->GetCentralityPercentile("ZNA");
 
   THnSparseD *hs = dynamic_cast<THnSparseD*>(fListUserOutputs->FindObject("hsReco"));
+  if (!hs)
+    return kTRUE;  // should not happen; make Coverity happen
 //=============================================================================
 
   AliPicoV0MC *pV0 = 0;
@@ -395,6 +397,9 @@ Bool_t AliAnalysisTaskEmcalJetV0CF::FillKineInfo()
   Double_t dZNA = fCentInfo->GetCentralityPercentile("ZNA");
 
   THnSparseD *hs = dynamic_cast<THnSparseD*>(fListUserOutputs->FindObject("hsKine"));
+  if (!hs) {
+    return kTRUE; // Should not happen; make Coverity happy
+  }
 //=============================================================================
 
   AliStack *pStack = 0;
index 232aad5..42266a7 100644 (file)
@@ -309,6 +309,7 @@ Bool_t AliAnalysisTaskEmcalJetV0Filter::Run()
   if (fPicoJetsClArrMC04) fPicoJetsClArrMC04->Delete();
 
   if (fPicoV0sClArr) fPicoV0sClArr->Delete();
+  else return kFALSE; // Should not happen, make Coverity happy
 
   if (!AliAnalysisTaskEmcalJet::Run()) return kFALSE;
 //=============================================================================
index d3e846b..5ccb2d1 100644 (file)
@@ -1244,7 +1244,10 @@ Int_t  AliAnalysisTaskJetCore::SelectTrigger(TList *list,Double_t minT,Double_t
      Int_t im2=0;
      for(int it = 0;it < aod->GetNumberOfTracks();++it){
       AliAODTrack *tr = dynamic_cast<AliAODTrack*>(aod->GetTrack(it));
-      if(!tr) AliFatal("Not a standard AOD");
+      if(!tr) {
+        AliFatal("Not a standard AOD");
+        return -1;
+      }
       Bool_t bGood = false;
       if(fFilterType == 0)bGood = true;
       else if(fFilterType == 1)bGood = tr->IsHybridTPCConstrainedGlobal();