From 7cca01eb9668521543f9fa5a35895730fcb6a223 Mon Sep 17 00:00:00 2001 From: rgrosso Date: Thu, 13 Dec 2012 17:30:54 +0000 Subject: [PATCH] fix for coverity 20257 and warning (id number ?) --- STEER/CDB/AliCDBStorage.cxx | 6 +++--- STEER/STEER/AliReconstruction.cxx | 4 ++++ 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/STEER/CDB/AliCDBStorage.cxx b/STEER/CDB/AliCDBStorage.cxx index 1ff59070537..2b036df58d3 100644 --- a/STEER/CDB/AliCDBStorage.cxx +++ b/STEER/CDB/AliCDBStorage.cxx @@ -506,11 +506,11 @@ AliCDBManager::DataType AliCDBStorage::GetDataType() const { //_____________________________________________________________________________ void AliCDBStorage::SetMirrorSEs(const char* mirrors) { // if the current storage is not of "alien" type, just issue a warning -// AliCDBGrid implements its own SetMirrorSEs method +// AliCDBGrid implements its own SetMirrorSEs method, classes for other storage types do not TString storageType = GetType(); if(storageType != "alien"){ - AliWarning(Form("The current storage is of type \"%s\" and cannot handle SEs. Setting skipped!",storageType.Data())); + AliWarning(Form("The current storage is of type \"%s\". Setting of SEs to \"%s\" skipped!",storageType.Data(),mirrors)); return; } AliError("We should never get here!! AliCDBGrid must have masked this virtual method!"); @@ -520,7 +520,7 @@ void AliCDBStorage::SetMirrorSEs(const char* mirrors) { //_____________________________________________________________________________ const char* AliCDBStorage::GetMirrorSEs() const { // if the current storage is not of "alien" type, just issue a warning -// AliCDBGrid implements its own GetMirrorSEs method +// AliCDBGrid implements its own GetMirrorSEs method, classes for other storage types do not TString storageType = GetType(); if(storageType != "alien"){ diff --git a/STEER/STEER/AliReconstruction.cxx b/STEER/STEER/AliReconstruction.cxx index 5ccdcc0d3a3..50890067919 100644 --- a/STEER/STEER/AliReconstruction.cxx +++ b/STEER/STEER/AliReconstruction.cxx @@ -1777,6 +1777,10 @@ void AliReconstruction::SlaveBegin(TTree*) // ESD layout template in CDB AliCDBManager* man = AliCDBManager::Instance(); AliCDBEntry* hltESDConfig = man->Get("HLT/Calib/esdLayout"); + if(!hltESDConfig){ + AliError(Form("Error getting \"HLT/Calib/esdLayout\"")); + return; + } AliESDEvent* pESDLayout=dynamic_cast(hltESDConfig->GetObject()); if (pESDLayout) { // init all internal variables from the list of objects -- 2.43.0