Coverity fixes
authorhaavard <haavard@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 20 Jan 2011 16:01:35 +0000 (16:01 +0000)
committerhaavard <haavard@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 20 Jan 2011 16:01:35 +0000 (16:01 +0000)
TPC/AliTPCPreprocessor.cxx
TPC/AliTPCSensorTempArray.cxx

index f0163c9..7f1ff31 100644 (file)
@@ -249,8 +249,11 @@ UInt_t AliTPCPreprocessor::Process(TMap* dcsAliasMap)
   TObject * status;
 
   UInt_t dcsResult=0;
-  if (!dcsAliasMap) dcsResult=1;
-  if (dcsAliasMap->GetEntries() == 0 ) dcsResult=1;  
+  if (!dcsAliasMap) { 
+     dcsResult=1;
+  } else {
+     if (dcsAliasMap->GetEntries() == 0 ) dcsResult=1;
+  }  
   status = new TParameter<int>("dcsResult",dcsResult);
   resultArray->Add(status);
 
@@ -1319,8 +1322,9 @@ UInt_t AliTPCPreprocessor::ExtractAltro(Int_t sourceFXS, TMap* dcsMap)
                                         fileNameEntry->GetString().Data());
         TFile *f = TFile::Open(fileName);
         if (!f) {
-          char message[40];
-         sprintf(message,"Error opening Altro configuration file, id = %d",id);
+         const int mess_length=40;
+          char message[mess_length];
+         snprintf(message,mess_length,"Error opening Altro configuration file, id = %d",id);
          Log (message);
          result =2;
          break;
index 1605f70..fc854f3 100644 (file)
@@ -123,8 +123,12 @@ void AliTPCSensorTempArray::ReadSensors(const char *dbEntry)
   // Read list of temperature sensors from text file
   //
   AliCDBEntry *entry = AliCDBManager::Instance()->Get(dbEntry);
+  if (!entry) {
+     AliWarning(Form("No OCDB entry  %s available\n",dbEntry));
+     return;
+  }        
   TTree *tree = (TTree*) entry->GetObject();
-  fSensors = AliTPCSensorTemp::ReadTree(tree);
+  if (tree) fSensors = AliTPCSensorTemp::ReadTree(tree);
 
 }