Coverity fixes
authorpulvir <pulvir@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 6 Apr 2011 09:57:02 +0000 (09:57 +0000)
committerpulvir <pulvir@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 6 Apr 2011 09:57:02 +0000 (09:57 +0000)
PWG2/RESONANCES/AliRsnExpression.cxx
PWG2/RESONANCES/extra/AliAnalysisTaskSigma1385.cxx

index 3d0d033..8c7e334 100644 (file)
@@ -101,6 +101,11 @@ Bool_t AliRsnExpression::Value(TObjArray &vars)
       AliError("Expression undefined.");
       return kFALSE;
    }
+   if (!fArg2) {
+      AliError("Argument 2 is required.");
+      return kFALSE;
+   }
+
 
 //   AliDebug(AliLog::kDebug,Form("fOperator %d",fOperator));
 
index 79fa002..9eb5b4c 100644 (file)
@@ -225,7 +225,10 @@ void AliAnalysisTaskSigma1385::UserExec(Option_t *)
    } else if (fAnalysisType == "AOD") {
       lAODevent = dynamic_cast<AliAODEvent*>(InputEvent());
       if (lAODevent) ncascades = lAODevent->GetNumberOfCascades();
-   } else {
+   } 
+   
+   // skip when both sources are unavailable
+   if (!lESDevent && !lAODevent) {
       Printf("ERROR: neither lESDevent nor lAODevent are available \n");
       return;
    }
@@ -639,7 +642,7 @@ void AliAnalysisTaskSigma1385::IsSelected(AliESDtrack *track)
    status  = (ULong_t)track->GetStatus();
    mom     = track->P();
    isTPC   = ((status & AliESDtrack::kTPCin)  != 0);
-   isITSSA = (!isTPC && (status & AliESDtrack::kITSrefit) != 0 && (status & AliESDtrack::kITSpureSA) == 0 && (status & AliESDtrack::kITSpid) != 0);
+   isITSSA = !isTPC && track->IsOn(AliESDtrack::kITSrefit) && !track->IsOn(AliESDtrack::kITSpureSA); //(!isTPC && (status & AliESDtrack::kITSrefit) != 0 && (status & AliESDtrack::kITSpureSA) == 0 && (status & AliESDtrack::kITSpid) != 0);
    isTOF   = (((status & AliESDtrack::kTOFout) != 0) && ((status & AliESDtrack::kTIME) != 0) /* && mom > TMath::Max(b1, b2)*/);