Coverity fixes
authorpulvir <pulvir@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 11 Apr 2011 09:28:42 +0000 (09:28 +0000)
committerpulvir <pulvir@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 11 Apr 2011 09:28:42 +0000 (09:28 +0000)
PWG2/RESONANCES/AliRsnCutPIDNSigma.cxx
PWG2/RESONANCES/AliRsnExpression.cxx
PWG2/RESONANCES/extra/AliAnalysisTaskSigma1385.cxx

index 121bd12..d300179 100644 (file)
@@ -167,7 +167,7 @@ void AliRsnCutPIDNSigma::Print(const Option_t *) const
 // Print information on this cut
 //
 
-   Char_t mom[200], det[10], match[200];
+   Char_t mom[200], det[100], match[200];
    
    if (fRejectOutside)
       snprintf(mom, 200, "Tracks are accepted only in the momentum range %.2f --> %.2f", fMomMin, fMomMax);
@@ -180,10 +180,10 @@ void AliRsnCutPIDNSigma::Print(const Option_t *) const
       snprintf(match, 200, "No check on track matching");
       
    switch (fDetector) {
-      case kITS: sprintf(det, "ITS"); break;
-      case kTPC: sprintf(det, "TPC"); break;
-      case kTOF: sprintf(det, "TOF"); break;
-      default  : sprintf(det, "undefined");
+      case kITS: snprintf(det, 3, "ITS"); break;
+      case kTPC: snprintf(det, 3, "TPC"); break;
+      case kTOF: snprintf(det, 3, "TOF"); break;
+      default  : snprintf(det, 3, "undefined");
    }
 
    AliInfo(Form("Cut name          : %s", GetName()));
index 3d0d033..943248d 100644 (file)
@@ -101,6 +101,10 @@ Bool_t AliRsnExpression::Value(TObjArray &vars)
       AliError("Expression undefined.");
       return kFALSE;
    }
+   if (fArg2 == 0) {
+      AliError("Needed second parameter");
+      return kFALSE;
+   }
 
 //   AliDebug(AliLog::kDebug,Form("fOperator %d",fOperator));
 
index 8496fec..7fc17f8 100644 (file)
@@ -278,6 +278,10 @@ void AliAnalysisTaskSigma1385::UserExec(Option_t *)
    }
 
    // initialize DB to current run
+   if (!lESDevent) {
+      Printf("ERROR: lESDevent not available \n");
+      return;
+   }
    Int_t run = lESDevent->GetRunNumber();
    if (run != fLastRun) {
       //cout << "Run = " << run << " -- LAST = " << fLastRun << endl;