Coverity fixes
authorhaavard <haavard@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 14 Dec 2010 15:31:45 +0000 (15:31 +0000)
committerhaavard <haavard@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 14 Dec 2010 15:31:45 +0000 (15:31 +0000)
TPC/AliTPCSensorTemp.cxx
TPC/AliTPCSensorTempArray.cxx

index 7f52bf0..167daca 100644 (file)
@@ -185,7 +185,7 @@ TClonesArray * AliTPCSensorTemp::ReadTree(TTree *tree,
   Int_t sector=0;
   char  type[100];
   char  side[100];
-  Int_t num=0;
+  UInt_t num=0;
   Int_t echa=0;
   //Double_t x=0;
   //Double_t y=0;
@@ -329,13 +329,14 @@ TClonesArray * AliTPCSensorTemp::ReadTree(TTree *tree,
       temp->SetZ(-240);
       }
 
-    if(temp->GetType()==5 && temp->GetSide()==0) {
+    if ( num < (sizeof(kTSz)/sizeof(kTSz[0]))) {
+      if(temp->GetType()==5 && temp->GetSide()==0) {
          temp->SetZ(kTSz[num]);
       }
-    if(temp->GetType()==5 && temp->GetSide()==1) {
+      if(temp->GetType()==5 && temp->GetSide()==1) {
          temp->SetZ(-kTSz[num]);
       }
-
+    }
 
   }
   return array;
index b1dda2c..1605f70 100644 (file)
@@ -49,10 +49,11 @@ AliTPCSensorTempArray::AliTPCSensorTempArray(Int_t run) : AliDCSSensorArray()
      
   AliCDBEntry *entry =
             AliCDBManager::Instance()->Get("TPC/Config/Temperature",run); 
-  TTree *tree = (TTree*) entry->GetObject();
-  fSensors = AliTPCSensorTemp::ReadTree(tree);
-  fSensors->BypassStreamer(kFALSE);
-  
+  if (entry) {
+    TTree *tree = (TTree*) entry->GetObject();
+    fSensors = AliTPCSensorTemp::ReadTree(tree);
+    fSensors->BypassStreamer(kFALSE);
+  }
 }
 //_____________________________________________________________________________
 AliTPCSensorTempArray::AliTPCSensorTempArray(UInt_t startTime, UInt_t endTime,