Coverity fixes
authormvl <marco.van.leeuwen@cern.ch>
Fri, 9 Jan 2015 15:56:53 +0000 (16:56 +0100)
committermvl <marco.van.leeuwen@cern.ch>
Fri, 9 Jan 2015 16:32:58 +0000 (17:32 +0100)
PWGJE/EMCALJetTasks/UserTasks/AliAnalysisTaskJetJTJT.cxx
PWGJE/FlavourJetTasks/AliAnalysisTaskSEPicoV0Maker.cxx

index f3efe7f..a690e74 100644 (file)
@@ -953,6 +953,7 @@ JTJTEfficiency::JTJTEfficiency():
        fInputRoot(NULL),
        fCentBinAxis(0x0)
 {
+  for (Int_t i=0; i < 3; i++) fEffDir[i] = 0;
 
 }
 
@@ -971,6 +972,7 @@ JTJTEfficiency::JTJTEfficiency(const JTJTEfficiency& obj) :
 {
        // copy constructor TODO: handling of pointer members
        JUNUSED(obj);
+  for (Int_t i=0; i < 3; i++) fEffDir[i] = obj.fEffDir[i];
 }
 
 JTJTEfficiency& JTJTEfficiency::operator=(const JTJTEfficiency& obj){
@@ -1067,8 +1069,8 @@ bool JTJTEfficiency::Load(){
        fInputRoot = TFile::Open( fInputRootName);
        //fInputRoot = new TFile( fInputRootName,"READ");
        if( !fInputRoot ) {
-               cout<< "J_ERROR : "<<fInputRootName <<" does not exist"<<endl;
-               gSystem->Exit(1);
+               cout << "J_ERROR : %s does not exist" << fInputRootName << endl;
+                return false;
        }
 
        //fEffDir[0] = (TDirectory*)fInputRoot->Get("EffRE");
@@ -1077,13 +1079,13 @@ bool JTJTEfficiency::Load(){
        //iif( fEffDir[0] && fEffDir[1] && fEffDir[2] )
        if( !fEffDir[2] )
        {
-               cout<< "J_ERROR : Directory EFF is not exist"<<endl;
-               gSystem->Exit(1);
+               cout << "J_ERROR : Directory EFF is not exist"<<endl;
+                return false;
        }
 
        fCentBinAxis = (TAxis*)fEffDir[2]->Get("CentralityBin");
        if( !fCentBinAxis ){
-               cout<< "J_ERROR : No CentralityBin in directory"<<endl;
+               cout << "J_ERROR : No CentralityBin in directory" << endl;
                gSystem->Exit(1);
        }
 
index 04e1c8a..28ef47e 100644 (file)
@@ -1585,6 +1585,11 @@ void AliAnalysisTaskSEPicoV0Maker::FillHistogramsMC()
 
   THnSparseD *hsV0 = dynamic_cast<THnSparseD*>(fOutputListMC->FindObject("hsV0"));
   THnSparseD *hsXi = dynamic_cast<THnSparseD*>(fOutputListMC->FindObject("hsXi"));
+
+  if (hsV0 == 0 || hsXi == 0) { // Keep Coverity happy
+    AliFatal("Cannot find hsV0 or hsXi; should not happen");
+    return;
+  }  // Keep Coverity happy
 //=============================================================================
 
   TParticle        *pESD = 0;