Fixes for coverity
authorcoppedis <Chiara.Oppedisano@to.infn.it>
Mon, 13 Oct 2014 09:14:44 +0000 (11:14 +0200)
committercoppedis <Chiara.Oppedisano@to.infn.it>
Mon, 13 Oct 2014 09:14:44 +0000 (11:14 +0200)
ZDC/AliZDC.cxx
ZDC/AliZDCDigitizer.cxx
ZDC/AliZDCQADataMakerRec.cxx
ZDC/AliZDCRawStream.cxx
ZDC/AliZDCReconstructor.cxx
ZDC/AliZDCTriggerProcessor.cxx

index 0e069a1..4a497b4 100644 (file)
@@ -774,35 +774,37 @@ Int_t AliZDC::Pedestal(Int_t Det, Int_t Quad, Int_t Res) const
   // Returns a pedestal for detector det, PM quad, channel with res.
   //
   // Getting calibration object for ZDC set
+  Float_t pedValue = 0.;
   AliCDBManager *man = AliCDBManager::Instance();
   AliCDBEntry  *entry = man->Get("ZDC/Calib/Pedestals");
+  AliZDCPedestals *calibPed = 0x0;
   if(!entry) AliFatal("No calibration data loaded!");  
-  AliZDCPedestals *calibPed = (AliZDCPedestals*) entry->GetObject();
-  //
-  if(!calibPed){
-    printf("\t No calibration object found for ZDC!");
-    return -1;
-  }
-  //
-  Int_t index=0, kNch=24;
-  if(Quad!=5){
-    if(Det==1)        index = Quad+kNch*Res;    // ZN1
-    else if(Det==2)   index = Quad+5+kNch*Res;          // ZP1
-    else if(Det==3)   index = Quad+9+kNch*Res; // ZEM
-    else if(Det==4)   index = Quad+12+kNch*Res; // ZN2
-    else if(Det==5)   index = Quad+17+kNch*Res; // ZP2
+  else{
+    calibPed = (AliZDCPedestals*) entry->GetObject();
+    //
+    if(!calibPed){
+      printf("\t No calibration object found for ZDC!");
+      return -1;
+    }
+    //
+    Int_t index=0, kNch=24;
+    if(Quad!=5){
+      if(Det==1)        index = Quad+kNch*Res;  // ZN1
+      else if(Det==2)   index = Quad+5+kNch*Res;        // ZP1
+      else if(Det==3)   index = Quad+9+kNch*Res; // ZEM
+      else if(Det==4)   index = Quad+12+kNch*Res; // ZN2
+      else if(Det==5)   index = Quad+17+kNch*Res; // ZP2
+    }
+    else index = (Det-1)/3+22+kNch*Res; // Reference PMs
+    //
+    //
+    Float_t meanPed = calibPed->GetMeanPed(index);
+    Float_t pedWidth = calibPed->GetMeanPedWidth(index);
+    pedValue = gRandom->Gaus(meanPed,pedWidth);
+    //
+    //printf("\t AliZDC::Pedestal - det(%d, %d) - Ped[%d] = %d\n",Det, Quad, index,(Int_t) pedValue); // Chiara debugging!
   }
-  else index = (Det-1)/3+22+kNch*Res; // Reference PMs
-  //
-  //
-  Float_t meanPed = calibPed->GetMeanPed(index);
-  Float_t pedWidth = calibPed->GetMeanPedWidth(index);
-  Float_t pedValue = gRandom->Gaus(meanPed,pedWidth);
-  //
-  //printf("\t AliZDC::Pedestal - det(%d, %d) - Ped[%d] = %d\n",Det, Quad, index,(Int_t) pedValue); // Chiara debugging!
-  
   
-
   return (Int_t) pedValue;
 }
 
@@ -847,12 +849,12 @@ AliZDCChMap* AliZDC::GetChMap() const
 {
 
   // Getting calibration object for ZDC
-
+  AliZDCChMap *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/ChMap");
   if(!entry) AliFatal("No calibration data loaded!");  
-
-  AliZDCChMap *calibdata = dynamic_cast<AliZDCChMap*> (entry->GetObject());
-  if(!calibdata) AliFatal("Wrong calibration object in calibration  file!");
-
+  else{
+    calibdata = dynamic_cast<AliZDCChMap*> (entry->GetObject());
+    if(!calibdata) AliFatal("Wrong calibration object in calibration  file!");
+  }
   return calibdata;
 }
index 7320900..2229a47 100644 (file)
@@ -832,10 +832,11 @@ AliZDCPedestals* AliZDCDigitizer::GetPedData() const
 
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/Pedestals");
   if(!entry) AliFatal("No calibration data loaded!");  
+  else{
+    AliZDCPedestals *calibdata = dynamic_cast<AliZDCPedestals*>  (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
 
-  AliZDCPedestals *calibdata = dynamic_cast<AliZDCPedestals*>  (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
-
-  return calibdata;
+    return calibdata;
+  }
 }
 
index 0c69643..79f8543 100644 (file)
@@ -883,7 +883,8 @@ void AliZDCQADataMakerRec::EndOfDetectorCycle(AliQAv1::TASKINDEX_t task, TObjArr
            h23->SetBinContent(6, h19->GetMean());
            h23->SetBinError(6, h19->GetRMS());
            //
-           h23->SetLineColor(kAzure+6); h22->SetLineWidth(2);
+           h23->SetLineColor(kAzure+6); 
+           h23->SetLineWidth(2);
          }
          else printf(" Warning: Raw TDC QA histo not found\n\n");
         }
index ba0c166..f332132 100644 (file)
@@ -997,9 +997,11 @@ AliZDCChMap* AliZDCRawStream::GetChMap() const
 
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/ChMap");
   if(!entry) AliFatal("No calibration data loaded!");  
+  else{
+   
+    AliZDCChMap *calibdata = dynamic_cast<AliZDCChMap*> (entry->GetObject());
+    if(!calibdata) AliFatal("Wrong calibration object in calibration  file!");
 
-  AliZDCChMap *calibdata = dynamic_cast<AliZDCChMap*> (entry->GetObject());
-  if(!calibdata) AliFatal("Wrong calibration object in calibration  file!");
-
-  return calibdata;
+    return calibdata;
+  }
 }
index 6ae77a6..a156861 100644 (file)
@@ -133,11 +133,12 @@ void AliZDCReconstructor::Init()
 
   AliCDBEntry *entry = AliCDBManager::Instance()->Get("GRP/Calib/LHCClockPhase"); 
   if (!entry) AliFatal("LHC clock-phase shift is not found in OCDB !");
-  AliLHCClockPhase *phaseLHC = (AliLHCClockPhase*)entry->GetObject();
-  // 4/2/2011 According to A. Di Mauro BEAM1 measurement is more reliable 
-  // than BEAM2 and therefore also than the average of the 2
-  fMeanPhase = phaseLHC->GetMeanPhaseB1();
-    
+  else{
+    AliLHCClockPhase *phaseLHC = (AliLHCClockPhase*)entry->GetObject();
+    // 4/2/2011 According to A. Di Mauro BEAM1 measurement is more reliable 
+    // than BEAM2 and therefore also than the average of the 2
+    fMeanPhase = phaseLHC->GetMeanPhaseB1();
+  }  
   if(fIsCalibrationMB==kFALSE)  
     AliInfo(Form("\n\n ***** ZDC reconstruction initialized for %s @ %1.0f + %1.0f GeV *****\n\n",
        beamType.Data(), fBeamEnergy, fBeamEnergy));
@@ -182,9 +183,10 @@ void AliZDCReconstructor::Init(TString beamType, Float_t beamEnergy)
 
   AliCDBEntry *entry = AliCDBManager::Instance()->Get("GRP/Calib/LHCClockPhase"); 
   if (!entry) AliFatal("LHC clock-phase shift is not found in OCDB !");
-  AliLHCClockPhase *phaseLHC = (AliLHCClockPhase*)entry->GetObject();
-  fMeanPhase = phaseLHC->GetMeanPhase();
-  
+  else{
+    AliLHCClockPhase *phaseLHC = (AliLHCClockPhase*)entry->GetObject();
+    fMeanPhase = phaseLHC->GetMeanPhase();
+  }
   fESDZDC = new AliESDZDC();
   
   AliInfo(Form("\n\n ***** ZDC reconstruction initialized for %s @ %1.0f + %1.0f GeV *****\n\n",
@@ -229,7 +231,7 @@ void AliZDCReconstructor::Reconstruct(TTree* digitsTree, TTree* clustersTree) co
   // -- Reading out-of-time signals (last kNch entries) for current event
   if(fPedSubMode==1){
     for(Int_t iDigit=kNch; iDigit<digNentries; iDigit++){
-       if(i<=kNch) ootDigi[i] = digitsTree->GetEntry(iDigit);
+       if(i<=kNch) ootDigi[i-1] = digitsTree->GetEntry(iDigit);
        else AliWarning(" Can't read more out of time values: index>kNch !!!\n");
        i++;
     }
@@ -1461,14 +1463,16 @@ AliZDCPedestals* AliZDCReconstructor::GetPedestalData() const
 {
 
   // Getting pedestal calibration object for ZDC set
-
+  AliZDCPedestals *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/Pedestals");
   if(!entry) AliFatal("No calibration data loaded!");
-  entry->SetOwner(kFALSE);
+  else{
+    entry->SetOwner(kFALSE);
 
-  AliZDCPedestals *calibdata = dynamic_cast<AliZDCPedestals*>  (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+    calibdata = dynamic_cast<AliZDCPedestals*>  (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
 
+  }
   return calibdata;
 }
 
@@ -1477,14 +1481,15 @@ AliZDCEnCalib* AliZDCReconstructor::GetEnergyCalibData() const
 {
 
   // Getting energy and equalization calibration object for ZDC set
-
+  AliZDCEnCalib *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/EnergyCalib");
   if(!entry) AliFatal("No calibration data loaded!");  
-  entry->SetOwner(kFALSE);
-
-  AliZDCEnCalib *calibdata = dynamic_cast<AliZDCEnCalib*> (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  else{
+    entry->SetOwner(kFALSE);
 
+    calibdata = dynamic_cast<AliZDCEnCalib*> (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  }
   return calibdata;
 }
 
@@ -1493,14 +1498,15 @@ AliZDCSaturationCalib* AliZDCReconstructor::GetSaturationCalibData() const
 {
 
   // Getting energy and equalization calibration object for ZDC set
-
+  AliZDCSaturationCalib *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/SaturationCalib");
   if(!entry) AliFatal("No calibration data loaded!");  
-  entry->SetOwner(kFALSE);
-
-  AliZDCSaturationCalib *calibdata = dynamic_cast<AliZDCSaturationCalib*> (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  else{
+    entry->SetOwner(kFALSE);
 
+    calibdata = dynamic_cast<AliZDCSaturationCalib*> (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  }
   return calibdata;
 }
 
@@ -1509,14 +1515,15 @@ AliZDCTowerCalib* AliZDCReconstructor::GetTowerCalibData() const
 {
 
   // Getting energy and equalization calibration object for ZDC set
-
+  AliZDCTowerCalib *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/TowerCalib");
   if(!entry) AliFatal("No calibration data loaded!");  
-  entry->SetOwner(kFALSE);
-
-  AliZDCTowerCalib *calibdata = dynamic_cast<AliZDCTowerCalib*> (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  else{
+    entry->SetOwner(kFALSE);
 
+    calibdata = dynamic_cast<AliZDCTowerCalib*> (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  }
   return calibdata;
 }
 
@@ -1525,14 +1532,15 @@ AliZDCMBCalib* AliZDCReconstructor::GetMBCalibData() const
 {
 
   // Getting energy and equalization calibration object for ZDC set
-
+  AliZDCMBCalib *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/MBCalib");
   if(!entry) AliFatal("No calibration data loaded!");  
-  entry->SetOwner(kFALSE);
-
-  AliZDCMBCalib *calibdata = dynamic_cast<AliZDCMBCalib*> (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  else{
+    entry->SetOwner(kFALSE);
 
+    calibdata = dynamic_cast<AliZDCMBCalib*> (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+  }
   return calibdata;
 }
 
@@ -1541,13 +1549,15 @@ AliZDCTDCCalib* AliZDCReconstructor::GetTDCCalibData() const
 {
 
   // Getting TDC object for ZDC 
-
+  AliZDCTDCCalib *calibdata = 0x0;
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Calib/TDCCalib");
   if(!entry) AliFatal("No calibration data loaded!");  
-  entry->SetOwner(kFALSE);
+  else{
+    entry->SetOwner(kFALSE);
 
-  AliZDCTDCCalib *calibdata = dynamic_cast<AliZDCTDCCalib*> (entry->GetObject());
-  if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
+    calibdata = dynamic_cast<AliZDCTDCCalib*> (entry->GetObject());
+    if(!calibdata)  AliFatal("Wrong calibration object in calibration  file!");
 
+  }
   return calibdata;
 }
index 4feb2c9..1f79e4f 100644 (file)
@@ -155,10 +155,10 @@ AliZDCTriggerParameters* AliZDCTriggerProcessor::GetTriggerParamFromOCDB() const
   // retrieving trigger parameter configuration form OCDB
   AliCDBEntry  *entry = AliCDBManager::Instance()->Get("ZDC/Trigger/");
   if(!entry) AliFatal("No calibration data loaded!");  
+  else{
+    AliZDCTriggerParameters *trigParam = dynamic_cast<AliZDCTriggerParameters*>  (entry->GetObject());
+    if(!trigParam)  AliFatal("Wrong calibration object in calibration  file!");
 
-  AliZDCTriggerParameters *trigParam = dynamic_cast<AliZDCTriggerParameters*>  (entry->GetObject());
-  if(!trigParam)  AliFatal("Wrong calibration object in calibration  file!");
-
-  return trigParam;
-  
+    return trigParam;
+  }
 }