Coverity FORWARD_NULL fixes
authorlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 8 Nov 2010 14:41:41 +0000 (14:41 +0000)
committerlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 8 Nov 2010 14:41:41 +0000 (14:41 +0000)
MUON/AliMUONClusterFinderMLEM.cxx
MUON/AliMUONClusterFinderPeakCOG.cxx
MUON/AliMUONClusterFinderPeakFit.cxx
MUON/AliMUONTrackerDataCompareDialog.cxx

index 4532938..66e6895 100644 (file)
@@ -1394,6 +1394,9 @@ Int_t AliMUONClusterFinderMLEM::FindLocalMaxima(TObjArray *pixArray, Int_t *loca
   Double_t xylim[4] = {999, 999, 999, 999};
 
   Int_t nPix = pixArray->GetEntriesFast();
+  
+  if ( nPix <= 0 ) return 0;
+  
   AliMUONPad *pixPtr = 0;
   for (Int_t ipix = 0; ipix < nPix; ++ipix) {
     pixPtr = (AliMUONPad*) pixArray->UncheckedAt(ipix);
index 9202e16..1291468 100644 (file)
@@ -693,6 +693,9 @@ Int_t AliMUONClusterFinderPeakCOG::FindLocalMaxima(TObjArray *pixArray, Int_t *l
   Double_t xylim[4] = {999, 999, 999, 999};
 
   Int_t nPix = pixArray->GetEntriesFast();
+  
+  if ( nPix <= 0 ) return 0;
+  
   AliMUONPad *pixPtr = 0;
   for (Int_t ipix = 0; ipix < nPix; ++ipix) {
     pixPtr = (AliMUONPad*) pixArray->UncheckedAt(ipix);
index e19b35b..c21c1ce 100644 (file)
@@ -809,6 +809,9 @@ Int_t AliMUONClusterFinderPeakFit::FindLocalMaxima(TObjArray *pixArray, Int_t *l
   Double_t xylim[4] = {999, 999, 999, 999};
 
   Int_t nPix = pixArray->GetEntriesFast();
+  
+  if ( nPix <= 0 ) return 0;
+
   AliMUONPad *pixPtr = 0;
   for (Int_t ipix = 0; ipix < nPix; ++ipix) {
     pixPtr = (AliMUONPad*) pixArray->UncheckedAt(ipix);
index e12d431..4f34286 100644 (file)
@@ -245,13 +245,16 @@ AliMUONTrackerDataCompareDialog::CompareData(const char* d1name,
     suffix = "ARD";
   }
   
-  TString basename = fBasename->GetText(); 
-  
-  AliMUONVTrackerData* d = CompareData(*d1,*d2,Form("%s:%s",basename.Data(),suffix.Data()),difffunction);
-  
-  AliMUONVTrackerDataMaker* dw = new AliMUONTrackerDataWrapper(d);
+  if ( difffunction ) 
+  {
+    TString basename = fBasename->GetText(); 
   
-  AliMUONPainterDataRegistry::Instance()->Register(dw);
+    AliMUONVTrackerData* d = CompareData(*d1,*d2,Form("%s:%s",basename.Data(),suffix.Data()),difffunction);
+    
+    AliMUONVTrackerDataMaker* dw = new AliMUONTrackerDataWrapper(d);
+    
+    AliMUONPainterDataRegistry::Instance()->Register(dw);
+  }
 }
 
 //______________________________________________________________________________