Fixing Coverity defects
authorlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 4 Nov 2010 16:19:22 +0000 (16:19 +0000)
committerlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 4 Nov 2010 16:19:22 +0000 (16:19 +0000)
MUON/AliMUONCDB.cxx
MUON/AliMUONTrackerData.cxx
MUON/AliMUONTrackerIO.cxx

index df989ade7c99a53148e0af011878f2870c5806dc..343d264fe71cc68f035fc2d96fe6002be536abf7 100644 (file)
@@ -1524,10 +1524,10 @@ AliMUONCDB::ShowConfig(Bool_t withStatusMap)
   
     statusMaker->SetLimits(*recoParam);
   
-    delete recoParam;
-  
     UInt_t mask = recoParam->PadGoodnessMask();
-  
+
+    delete recoParam;
+    
     const Bool_t deferredInitialization = kFALSE;
   
     statusMapMaker = new AliMUONPadStatusMapMaker(*cd,mask,deferredInitialization);
index eb6215fffbdeda0a6db58a1db4623b9af9b91d56..ff0c925b28649e70fa980af921a6e6c536d3ca62 100644 (file)
@@ -1601,7 +1601,7 @@ AliMUONTrackerData::Merge(TCollection* list)
   while ( ( o = next() ) )
   {
     const AliMUONTrackerData* data = dynamic_cast<const AliMUONTrackerData*>(o);
-    if (!o)
+    if (!data)
     {
       AliError(Form("Object named %s is not an AliMUONTrackerData ! Skipping it",
                     o->GetName()));
index c0def9fba7aec6089d44687a35cc558624825918..cf3a39eb14621bdceb90cb976b1db3a2bb6c6c12 100644 (file)
@@ -484,6 +484,9 @@ AliMUONTrackerIO::DecodeCapacitances(const char* data, AliMUONVStore& capaStore)
       }      
       continue;
     }
+    
+    if (!param) continue;
+    
     Int_t channel;
     Float_t capaValue;
     Float_t injectionGain;