From ba05d2456da27f148a5952470b8054de819d9a59 Mon Sep 17 00:00:00 2001
From: jthaeder <jthaeder@f7af4fe6-9843-0410-8265-dc069ae4e863>
Date: Tue, 1 Mar 2011 22:47:42 +0000
Subject: [PATCH] coverity fixes

---
 HLT/JET/cone/AliHLTJETConeFinder.cxx | 39 +++++++++++++++++++++++-----
 1 file changed, 33 insertions(+), 6 deletions(-)

diff --git a/HLT/JET/cone/AliHLTJETConeFinder.cxx b/HLT/JET/cone/AliHLTJETConeFinder.cxx
index 06b4916905f..62f9b7c0aa0 100644
--- a/HLT/JET/cone/AliHLTJETConeFinder.cxx
+++ b/HLT/JET/cone/AliHLTJETConeFinder.cxx
@@ -149,12 +149,16 @@ void AliHLTJETConeFinder::Reset() {
 Bool_t AliHLTJETConeFinder::ProcessEvent() {
   // see header file for class documentation
 
-  // -- Pick up jet reader
-  AliHLTJETReader *reader = dynamic_cast<AliHLTJETReader*> (fReader);
-
   // -- Reset
   Reset();
 
+  // -- Pick up jet reader
+  AliHLTJETReader *reader = dynamic_cast<AliHLTJETReader*> (fReader);
+  if ( !reader ) {
+    HLTError("Error getting reader.");
+    return kFALSE;  
+  }
+
   // -- Fill Grid
   if ( !reader->FillGrid() ){
     HLTError("Error filling grid.");
@@ -222,6 +226,17 @@ Int_t AliHLTJETConeFinder::FindConeLeading() {
 
   // -- Pick up jet reader
   AliHLTJETReader *reader = dynamic_cast<AliHLTJETReader*> (fReader);
+  if ( !reader ) {
+    HLTError("Error getting reader.");
+    return -EINPROGRESS;
+  }
+
+  // -- Pick up jet header
+  AliHLTJETConeHeader* header = dynamic_cast<AliHLTJETConeHeader*> (fHeader);
+  if ( !header ) {
+    HLTError("Error getting header.");
+    return -EINPROGRESS;
+  }
 
   // -- Pick up jet canidates
   TClonesArray* jetCandidates = reader->GetJetCandidates();
@@ -234,7 +249,7 @@ Int_t AliHLTJETConeFinder::FindConeLeading() {
     
     // -- Use leading seed only
     //    Keep index 0, remove the others
-    if ( (dynamic_cast<AliHLTJETConeHeader*> (fHeader))->GetUseLeading() ) {
+    if ( header->GetUseLeading() ) {
       
       for ( Int_t iter = 1; iter < reader->GetNJetCandidates(); iter++ )
 	jetCandidates->RemoveAt(iter);
@@ -258,7 +273,11 @@ Int_t AliHLTJETConeFinder::FindConeJets() {
 
   // -- Pick up jet reader
   AliHLTJETReader *reader = dynamic_cast<AliHLTJETReader*> (fReader);
-   
+  if ( !reader ) {
+    HLTError("Error getting reader.");
+    return -EINPROGRESS;
+  }
+
   // -- Pick up jet canidates
   TClonesArray* jetCandidates = reader->GetJetCandidates();
 
@@ -298,10 +317,18 @@ Int_t AliHLTJETConeFinder::FillConeJets() {
 
   // -- Pick up jet reader
   AliHLTJETReader *reader = dynamic_cast<AliHLTJETReader*> (fReader);
+  if ( !reader ) {
+    HLTError("Error getting reader.");
+    return -EINPROGRESS;
+  }
 
   // -- Pick up jet header
   AliHLTJETConeHeader *header = dynamic_cast<AliHLTJETConeHeader*> (fHeader);
-  
+  if ( !header ) {
+    HLTError("Error getting header.");
+    return -EINPROGRESS;
+  }
+
   // -- Get jet canidates
   TClonesArray* jetCandidates = reader->GetJetCandidates();
   
-- 
2.43.5