fix coverity #24651 #24652
authorgconesab <gustavo.conesa.balbastre@cern.ch>
Thu, 30 Oct 2014 13:08:32 +0000 (14:08 +0100)
committergconesab <gustavo.conesa.balbastre@cern.ch>
Thu, 30 Oct 2014 13:17:44 +0000 (14:17 +0100)
PWGGA/CaloTrackCorrelations/AliAnaElectron.cxx
PWGGA/CaloTrackCorrelations/AliAnaPhoton.cxx
PWGGA/CaloTrackCorrelations/AliAnaPhotonConvInCalo.cxx

index e7008ac..760a55b 100755 (executable)
@@ -1405,13 +1405,21 @@ void  AliAnaElectron::MakeAnalysisFillHistograms()
     if(GetReader()->ReadStack())
     {
       stack =  GetMCStack() ;
-      if ( !stack )       AliFatal("Stack not available, is the MC handler called? STOP");
+      if ( !stack )
+      {
+        AliFatal("Stack not available, is the MC handler called? STOP");
+        return;
+      }
     }
     else if(GetReader()->ReadAODMCParticles())
     {
       //Get the list of MC particles
       mcparticles = GetReader()->GetAODMCParticles();
-      if ( !mcparticles ) AliFatal("Standard MCParticles not available! STOP");
+      if ( !mcparticles )
+      {
+        AliFatal("Standard MCParticles not available! STOP");
+        return;
+      }
     }
   }// is data and MC
   
@@ -1470,7 +1478,7 @@ void  AliAnaElectron::MakeAnalysisFillHistograms()
       
       Float_t eprim   = 0;
       //Float_t ptprim  = 0;
-      if(GetReader()->ReadStack())
+      if( GetReader()->ReadStack() )
       {
         if(label >=  stack->GetNtrack())
         {
@@ -1489,22 +1497,15 @@ void  AliAnaElectron::MakeAnalysisFillHistograms()
         //ptprim  = primary->Pt();
         
       }
-      else if(GetReader()->ReadAODMCParticles())
+      else if( GetReader()->ReadAODMCParticles() )
       {
-        //Check which is the input
-        if(ph->GetInputFileIndex() == 0)
+        if(label >=  mcparticles->GetEntriesFast())
         {
-          if(!mcparticles) continue;
-          
-          if(label >=  mcparticles->GetEntriesFast())
-          {
-            AliDebug(1,Form("*** large label ***:  label %d, n tracks %d",label, mcparticles->GetEntriesFast()));
-            continue ;
-          }
-          //Get the particle
-          aodprimary = (AliAODMCParticle*) mcparticles->At(label);
-          
+          AliDebug(1,Form("*** large label ***:  label %d, n tracks %d",label, mcparticles->GetEntriesFast()));
+          continue ;
         }
+        //Get the particle
+        aodprimary = (AliAODMCParticle*) mcparticles->At(label);
         
         if(!aodprimary)
         {
index 63cb027..e7a5f5f 100755 (executable)
@@ -348,7 +348,11 @@ void AliAnaPhoton::FillAcceptanceHistograms()
   if( GetReader()->ReadStack() )
   {
     stack = GetMCStack();
-    if(!stack ) return;
+    if( !stack )
+    {
+      AliFatal("Stack not available, is the MC handler called? STOP");
+      return;
+    }
     nprim = stack->GetNtrack();
   }
   
@@ -357,7 +361,11 @@ void AliAnaPhoton::FillAcceptanceHistograms()
   if( GetReader()->ReadAODMCParticles() )
   {
     mcparticles = GetReader()->GetAODMCParticles();
-    if( !mcparticles ) return;
+    if( !mcparticles )
+    {
+      AliFatal("Standard MCParticles not available!");
+      return;
+    }
     nprim = mcparticles->GetEntriesFast();
   }
   
index 1b69c75..761c41f 100755 (executable)
@@ -734,14 +734,20 @@ void  AliAnaPhotonConvInCalo::MakeAnalysisFillHistograms()
     {
       stack =  GetMCStack() ;
       if(!stack)
+      {
         AliFatal("Stack not available, is the MC handler called? STOP");
+        return;
+      }
     }
     else if(GetReader()->ReadAODMCParticles())
     {
       //Get the list of MC particles
       mcparticles = GetReader()->GetAODMCParticles();
       if(!mcparticles)
+      {
         AliFatal("Standard MCParticles not available!");
+        return;
+      }
     }
   }// is data and MC
   
@@ -804,22 +810,15 @@ void  AliAnaPhotonConvInCalo::MakeAnalysisFillHistograms()
         }
         else if(GetReader()->ReadAODMCParticles())
         {
-          //Check which is the input
-          if(ph->GetInputFileIndex() == 0)
+          if(label >=  mcparticles->GetEntriesFast())
           {
-            if(!mcparticles) continue;
-            
-            if(label >=  mcparticles->GetEntriesFast())
-            {
-              AliDebug(2,Form("*** large label ***:  label %d, n tracks %d",label, mcparticles->GetEntriesFast()));
-              continue ;
-            }
-            
-            //Get the particle
-            aodprimary = (AliAODMCParticle*) mcparticles->At(label);
-            
+            AliDebug(2,Form("*** large label ***:  label %d, n tracks %d",label, mcparticles->GetEntriesFast()));
+            continue ;
           }
           
+          //Get the particle
+          aodprimary = (AliAODMCParticle*) mcparticles->At(label);
+          
           if(!aodprimary)
           {
             AliDebug(2,Form("*** no primary ***:  label %d", label));