coverity
authorjotwinow <jotwinow@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sat, 4 Jun 2011 18:55:18 +0000 (18:55 +0000)
committerjotwinow <jotwinow@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sat, 4 Jun 2011 18:55:18 +0000 (18:55 +0000)
PWG1/AliAnalysisTaskV0QA.cxx
PWG1/TPC/AliAnalysisNoiseTPC.cxx
PWG1/TPC/AliGenKinkInfo.cxx
PWG1/TPC/AliPerformanceEff.cxx
PWG1/TPC/AliTreeDraw.cxx

index 6ff9662..083d70d 100644 (file)
@@ -1554,7 +1554,7 @@ void AliAnalysisTaskV0QA::InspectListOfV0s(){
 
     AliESDtrack* trackPosTest = fESD->GetTrack(fV0MIs->GetPindex());
     AliESDtrack* trackNegTest = fESD->GetTrack(fV0MIs->GetNindex());
-
+     
 
     if ( trackPosTest->GetSign() == trackNegTest->GetSign()){
      continue;
@@ -1585,6 +1585,9 @@ void AliAnalysisTaskV0QA::InspectListOfV0s(){
 
     }
 
+    if(!trackPos) return;
+    if(!trackNeg) return;
+
     Int_t labelNeg=TMath::Abs(trackNeg->GetLabel());
     if(labelNeg > fStack->GetNtrack() ) continue;
     TParticle * particleNeg= fStack->Particle(labelNeg);
index 6b97e88..9594e2a 100644 (file)
@@ -209,8 +209,8 @@ void AliAnalysisNoiseTPC::UserExec(Option_t *)
     if (track->GetKinkIndex(0) > 0) continue;
     
     UInt_t status = track->GetStatus();
-    if ((status&AliESDtrack::kITSrefit)==1) continue; // explicitly ask for tracks without ITS refit
-    if ((status&AliESDtrack::kTPCrefit)==0) continue;
+    if ((status&AliESDtrack::kITSrefit)==AliESDtrack::kITSrefit) continue; // explicitly ask for tracks without ITS refit
+    if ((status&AliESDtrack::kTPCrefit)!=AliESDtrack::kTPCrefit) continue;
     
     if (track->GetTPCsignal() > 10) continue;          // explicitly ask for tracks without dE/dx
     
index 7049eec..202a742 100644 (file)
@@ -124,7 +124,8 @@ void AliGenKinkInfo::Update()
   p1[0] = pdaughter.Px();
   p1[1] = pdaughter.Py();
   p1[2] = pdaughter.Pz();
-  Double_t sign = (pdaughter.GetPDG()->Charge()>0)? -1:1;
+  Double_t sign = 0;
+  sign = (pdaughter.GetPDG()->Charge()>0)? -1:1;
   AliHelix dhelix1(x1,p1,sign);
   //
   //
index 162544c..a1b5ae6 100644 (file)
@@ -189,6 +189,13 @@ void AliPerformanceEff::ProcessTPC(AliMCEvent* const mcEvent, AliESDEvent *const
 {\r
   // Fill TPC only efficiency comparison information \r
   if(!esdEvent) return;\r
+  if(!mcEvent) return;\r
+\r
+  AliStack *stack = mcEvent->Stack();\r
+  if (!stack) {\r
+    AliDebug(AliLog::kError, "Stack not available");\r
+    return;\r
+  }\r
 \r
   Int_t *labelsRec =  NULL;\r
   labelsRec =  new Int_t[esdEvent->GetNumberOfTracks()];\r
@@ -202,6 +209,7 @@ void AliPerformanceEff::ProcessTPC(AliMCEvent* const mcEvent, AliESDEvent *const
   Int_t *labelsAllRec =  NULL;\r
   labelsAllRec =  new Int_t[esdEvent->GetNumberOfTracks()];\r
   if(!labelsAllRec) { \r
+     delete  [] labelsRec;\r
      Printf("Cannot create labelsAllRec");\r
      return;\r
   }\r
@@ -231,14 +239,6 @@ void AliPerformanceEff::ProcessTPC(AliMCEvent* const mcEvent, AliESDEvent *const
   // \r
   // MC histograms for efficiency studies\r
   //\r
-  if(!mcEvent) return;\r
\r
-  AliStack *stack = mcEvent->Stack();\r
-  if (!stack) {\r
-    AliDebug(AliLog::kError, "Stack not available");\r
-    return;\r
-  }\r
-\r
   Int_t nPart  = stack->GetNtrack();\r
   //Int_t nPart  = stack->GetNprimary();\r
   for (Int_t iMc = 0; iMc < nPart; ++iMc) \r
@@ -316,22 +316,23 @@ void AliPerformanceEff::ProcessTPCSec(AliMCEvent* const mcEvent, AliESDEvent *co
 \r
   if(!esdEvent) return;\r
 \r
-  Int_t *labelsRec =  NULL;\r
-  labelsRec =  new Int_t[esdEvent->GetNumberOfTracks()];\r
-  if(!labelsRec) \r
+  Int_t *labelsRecSec =  NULL;\r
+  labelsRecSec =  new Int_t[esdEvent->GetNumberOfTracks()];\r
+  if(!labelsRecSec) \r
   {\r
-     Printf("Cannot create labelsRec");\r
+     Printf("Cannot create labelsRecSec");\r
      return;\r
   }\r
-  for(Int_t i=0;i<esdEvent->GetNumberOfTracks();i++) { labelsRec[i] = 0; }\r
+  for(Int_t i=0;i<esdEvent->GetNumberOfTracks();i++) { labelsRecSec[i] = 0; }\r
 \r
-  Int_t *labelsAllRec =  NULL;\r
-  labelsAllRec =  new Int_t[esdEvent->GetNumberOfTracks()];\r
-  if(!labelsAllRec) { \r
-     Printf("Cannot create labelsAllRec");\r
+  Int_t *labelsAllRecSec =  NULL;\r
+  labelsAllRecSec =  new Int_t[esdEvent->GetNumberOfTracks()];\r
+  if(!labelsAllRecSec) { \r
+     delete [] labelsRecSec;\r
+     Printf("Cannot create labelsAllRecSec");\r
      return;\r
   }\r
-  for(Int_t i=0;i<esdEvent->GetNumberOfTracks();i++) { labelsAllRec[i] = 0; }\r
+  for(Int_t i=0;i<esdEvent->GetNumberOfTracks();i++) { labelsAllRecSec[i] = 0; }\r
 \r
   // loop over rec. tracks\r
   AliESDtrack *track=0;\r
@@ -346,12 +347,12 @@ void AliPerformanceEff::ProcessTPCSec(AliMCEvent* const mcEvent, AliESDEvent *co
     if(!fUseKinkDaughters && track->GetKinkIndex(0) > 0) continue;\r
 \r
     Int_t label = TMath::Abs(track->GetLabel()); \r
-    labelsAllRec[multAll]=label;\r
+    labelsAllRecSec[multAll]=label;\r
     multAll++;\r
 \r
     // TPC only\r
     if(IsRecTPC(track) != 0) {\r
-      labelsRec[multRec]=label;\r
+      labelsRecSec[multRec]=label;\r
       multRec++;\r
     }\r
   }\r
@@ -389,7 +390,7 @@ void AliPerformanceEff::ProcessTPCSec(AliMCEvent* const mcEvent, AliESDEvent *co
     for(Int_t iRec=0; iRec<multAll; ++iRec) \r
     {\r
       // check findable\r
-      if(iMc == labelsAllRec[iRec]) \r
+      if(iMc == labelsAllRecSec[iRec]) \r
       {\r
         findable = IsFindable(mcEvent,iMc);\r
        break;\r
@@ -400,7 +401,7 @@ void AliPerformanceEff::ProcessTPCSec(AliMCEvent* const mcEvent, AliESDEvent *co
     for(Int_t iRec=0; iRec<multRec; ++iRec) \r
     {\r
       // check reconstructed\r
-      if(iMc == labelsRec[iRec]) \r
+      if(iMc == labelsRecSec[iRec]) \r
       {\r
         recStatus = kTRUE;\r
         break;\r
@@ -440,8 +441,8 @@ void AliPerformanceEff::ProcessTPCSec(AliMCEvent* const mcEvent, AliESDEvent *co
   }\r
   }\r
 \r
-  if(labelsRec) delete [] labelsRec; labelsRec = 0;\r
-  if(labelsAllRec) delete [] labelsAllRec; labelsAllRec = 0;\r
+  if(labelsRecSec) delete [] labelsRecSec; labelsRecSec = 0;\r
+  if(labelsAllRecSec) delete [] labelsAllRecSec; labelsAllRecSec = 0;\r
 }\r
 \r
 \r
@@ -452,6 +453,15 @@ void AliPerformanceEff::ProcessTPCITS(AliMCEvent* const mcEvent, AliESDEvent *co
 {\r
   // Fill efficiency comparison information\r
 \r
+  if(!esdEvent) return;\r
+  if(!mcEvent) return;\r
+\r
+  AliStack *stack = mcEvent->Stack();\r
+  if (!stack) {\r
+    AliDebug(AliLog::kError, "Stack not available");\r
+    return;\r
+  }\r
+\r
   Int_t *labelsRecTPCITS =  NULL;\r
   labelsRecTPCITS =  new Int_t[esdEvent->GetNumberOfTracks()];\r
   if(!labelsRecTPCITS) \r
@@ -464,6 +474,7 @@ void AliPerformanceEff::ProcessTPCITS(AliMCEvent* const mcEvent, AliESDEvent *co
   Int_t *labelsAllRecTPCITS =  NULL;\r
   labelsAllRecTPCITS =  new Int_t[esdEvent->GetNumberOfTracks()];\r
   if(!labelsAllRecTPCITS) { \r
+     delete [] labelsRecTPCITS;\r
      Printf("Cannot create labelsAllRecTPCITS");\r
      return;\r
   }\r
@@ -491,14 +502,6 @@ void AliPerformanceEff::ProcessTPCITS(AliMCEvent* const mcEvent, AliESDEvent *co
   // \r
   // MC histograms for efficiency studies\r
   //\r
-  if(!mcEvent) return;\r
\r
-  AliStack *stack = mcEvent->Stack();\r
-  if (!stack) {\r
-    AliDebug(AliLog::kError, "Stack not available");\r
-    return;\r
-  }\r
-\r
   //Int_t nPart  = stack->GetNtrack();\r
   Int_t nPart  = stack->GetNprimary();\r
   for (Int_t iMc = 0; iMc < nPart; ++iMc) \r
@@ -563,6 +566,13 @@ void AliPerformanceEff::ProcessConstrained(AliMCEvent* const mcEvent, AliESDEven
 {\r
   // Process comparison information \r
   if(!esdEvent) return;\r
+  if(!mcEvent) return;\r
+\r
+  AliStack *stack = mcEvent->Stack();\r
+  if (!stack) {\r
+    AliDebug(AliLog::kError, "Stack not available");\r
+    return;\r
+  }\r
 \r
   Int_t *labelsRecConstrained =  NULL;\r
   labelsRecConstrained =  new Int_t[esdEvent->GetNumberOfTracks()];\r
@@ -576,6 +586,7 @@ void AliPerformanceEff::ProcessConstrained(AliMCEvent* const mcEvent, AliESDEven
   Int_t *labelsAllRecConstrained =  NULL;\r
   labelsAllRecConstrained =  new Int_t[esdEvent->GetNumberOfTracks()];\r
   if(!labelsAllRecConstrained) { \r
+     delete [] labelsRecConstrained;\r
      Printf("Cannot create labelsAllRecConstrained");\r
      return;\r
   }\r
@@ -604,13 +615,7 @@ void AliPerformanceEff::ProcessConstrained(AliMCEvent* const mcEvent, AliESDEven
   // \r
   // MC histograms for efficiency studies\r
   //\r
-  if(!mcEvent) return;\r
  \r
-  AliStack *stack = mcEvent->Stack();\r
-  if (!stack) {\r
-    AliDebug(AliLog::kError, "Stack not available");\r
-    return;\r
-  }\r
 \r
   //Int_t nPart  = stack->GetNtrack();\r
   Int_t nPart  = stack->GetNprimary();\r
index 0f832f1..7439b8e 100644 (file)
@@ -632,8 +632,10 @@ TString* AliTreeDraw::FitPlane(const char* drawCommand, const char* formula, con
       else  centries = fTree->Draw(drawStr.Data(), cutStr.Data(), "goff", stop-start,start);
       
       if (entries != centries) { 
+        delete[] *values;
         return new TString("An ERROR has occured during fitting!");
       }
+      
       values[i] = new Double_t[entries];
       memcpy(values[i],  fTree->GetV1(), entries*sizeof(Double_t)); 
    }