Coverity fixes
authorlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 4 Dec 2011 10:35:42 +0000 (10:35 +0000)
committerlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 4 Dec 2011 10:35:42 +0000 (10:35 +0000)
MUON/AliMUON2DMap.cxx
MUON/AliMUONClusterSplitterMLEM.cxx
MUON/AliMUONClusterStoreV2.cxx
MUON/AliMUONPadStatusMaker.cxx
MUON/AliMUONTrackerQAChecker.cxx
MUON/AliMUONTrackerQAChecker.h

index ff96bec..1955138 100644 (file)
@@ -75,8 +75,11 @@ AliMUON2DMap&
 AliMUON2DMap::operator=(const AliMUON2DMap& other)
 {
 /// Assignment operator
-  *fMap = *other.fMap;
-  fOptimizeForDEManu = other.fOptimizeForDEManu;
+  if ( this != &other )
+  {
+    *fMap = *other.fMap;
+    fOptimizeForDEManu = other.fOptimizeForDEManu;
+  }
   return *this;
 }
 
index feca1e2..53ba85d 100644 (file)
@@ -233,7 +233,11 @@ AliMUONClusterSplitterMLEM::Fcn1(const AliMUONCluster& cluster,
     delta /= pad->Charge(); 
     chi2 += delta;
   } // for (Int_t j=0;
-  if (iflag == 0) qAver = qTot / npads;
+  if (iflag == 0 && npads) qAver = qTot / npads;
+  if (!npads)
+  {
+    AliError(Form("Got npads=0. Please check"));
+  }
   f = chi2 / qAver;
 }
 
index e4c0552..2a7d76b 100644 (file)
@@ -69,10 +69,13 @@ AliMUONClusterStoreV2::AliMUONClusterStoreV2(const AliMUONClusterStoreV2& store)
 AliMUONClusterStoreV2& AliMUONClusterStoreV2::operator=(const AliMUONClusterStoreV2& store)
 {
   /// Assignment operator
-  fClusters = new TClonesArray(*(store.fClusters));
-  fMap = 0x0;
-  fMapped = kFALSE;
-  if (store.fMapped) ReMap();
+  if ( this != &store )
+  {
+    fClusters = new TClonesArray(*(store.fClusters));
+    fMap = 0x0;
+    fMapped = kFALSE;
+    if (store.fMapped) ReMap();
+  }
   return *this;
 }
 
index 67acc13..59d4c9f 100644 (file)
@@ -756,6 +756,12 @@ AliMUONPadStatusMaker::Report(UInt_t mask)
     }
   }
   
+  if (!nPads) 
+  {
+    AliError("Got no pad from the iterator ?! That's not normal. Please check !");
+    return;
+  }
+  
   TString msg;
   Int_t ntimes;
   
index a8b0773..e05e80b 100644 (file)
@@ -238,13 +238,6 @@ AliMUONTrackerQAChecker::~AliMUONTrackerQAChecker()
        /// dtor
 }
 
-//__________________________________________________________________
-AliMUONTrackerQAChecker::AliMUONTrackerQAChecker(const AliMUONTrackerQAChecker& qac) : 
-    AliMUONVQAChecker(qac) 
-{
-       /// copy ctor 
-}   
-
 //______________________________________________________________________________
 AliMUONVQAChecker::ECheckCode*
 AliMUONTrackerQAChecker::CheckRecPoints(TObjArray ** list, const AliMUONRecoParam* /*recoParam*/)
index 290d48f..c0a6673 100644 (file)
@@ -19,7 +19,6 @@ class AliMUONTrackerQAChecker: public AliMUONVQAChecker {
 
 public:
   AliMUONTrackerQAChecker();
-  AliMUONTrackerQAChecker(const AliMUONTrackerQAChecker& qac);
   virtual ~AliMUONTrackerQAChecker();
 
   virtual AliMUONVQAChecker::ECheckCode* CheckRaws(TObjArray** list, const AliMUONRecoParam* recoParam);