Coverity corrections
authorgconesab <gconesab@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 15 Oct 2010 14:04:08 +0000 (14:04 +0000)
committergconesab <gconesab@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 15 Oct 2010 14:04:08 +0000 (14:04 +0000)
EMCAL/AliEMCALCalibHistoProducer.cxx
EMCAL/AliEMCALTracker.cxx

index 4522fb4b8729f33174f18ffed885e16a9101af33..58c02b5fadf304010f6b70d9f916732e98d6a8ee 100644 (file)
@@ -147,14 +147,15 @@ void AliEMCALCalibHistoProducer::Init()
   // Checks existence of histograms which might have been left
   // from the previous runs to continue their filling
   fHistoFile =  new TFile(fHistoFileName,"update");
-  char hname[128];
+  const Int_t buffersize = 128;
+  char hname[buffersize];
   Int_t nRow =  fNCellsPhi ;
 
   for(Int_t supermodule=0; supermodule<fNSuperModules; supermodule++) {
     //Check installed supermodules
     if(fSMInstalled[supermodule]==kFALSE) continue;
     //Check created profiles
-    sprintf(hname,"mod%d",supermodule);
+    snprintf(hname,buffersize,"mod%d",supermodule);
     TProfile* prof = (TProfile*)fHistoFile->Get(hname);
     if(prof)
       fAmpProf[supermodule]=prof;
@@ -163,10 +164,10 @@ void AliEMCALCalibHistoProducer::Init()
     if(supermodule > 10) nRow = fNCellsPhiHalfSM ; //Supermodules 11 and 12 are half supermodules
     for(Int_t column=0; column<fNCellsEta; column++) {
       for(Int_t row=0; row<nRow; row++) {
-       sprintf(hname,"mod%dcol%drow%d",supermodule,column,row);
-       TH1F* hist = (TH1F*)fHistoFile->Get(hname);
-       if(hist) 
-         fAmpHisto[supermodule][column][row]=hist;
+        snprintf(hname,buffersize,"mod%dcol%drow%d",supermodule,column,row);
+        TH1F* hist = (TH1F*)fHistoFile->Get(hname);
+        if(hist) 
+          fAmpHisto[supermodule][column][row]=hist;
       }
     }
   }
@@ -183,7 +184,8 @@ void AliEMCALCalibHistoProducer::Run()
   
 //   TH1F* gHighGain = 0;
 //   TH1F* gLowGain = 0;
-  char hname[128];
+  const Int_t buffersize=128;
+  char hname[buffersize];
   Int_t iEvent = 0;
   Int_t runNum = 0;
   Int_t nProfFreq = 1000; //Number of events with which a bin of the TProfile if filled
@@ -223,11 +225,11 @@ void AliEMCALCalibHistoProducer::Run()
        
        //Check if histogram/profile already exist, if not create it.
        if(!fAmpHisto[mod][col][row]) {
-         sprintf(hname,"mod%dcol%drow%d",mod,col,row);
+         snprintf(hname,buffersize,"mod%dcol%drow%d",mod,col,row);
          fAmpHisto[mod][col][row] = new TH1F(hname,hname,1024,-0.5,1023.);
        }
        if(!fAmpProf[mod]) {
-         sprintf(hname,"mod%d",mod);
+         snprintf(hname,buffersize,"mod%d",mod);
          fAmpProf[mod] = new TProfile(hname,hname,nEvtBins,0.,nEvtBins);
        }
                
index 98c999c09f2301f56cbf54101e182da1721ed570..c381ff377d464098ad9e12756e0f583a1f331210 100644 (file)
@@ -161,16 +161,17 @@ void AliEMCALTracker::InitParameters()
   if(!recParam){
     AliFatal("Reconstruction parameters for EMCAL not set!");
   }
-  
-  fCutX =  recParam->GetTrkCutX();
-  fCutY =  recParam->GetTrkCutY();
-  fCutZ =  recParam->GetTrkCutZ();
-  fMaxDist =  recParam->GetTrkCutR();
-  fCutAngle =  recParam->GetTrkCutAngle();
-  fCutAlphaMin =  recParam->GetTrkCutAlphaMin();
-  fCutAlphaMax =  recParam->GetTrkCutAlphaMax();
-  fCutNITS = recParam->GetTrkCutNITS();
-  fCutNTPC = recParam->GetTrkCutNTPC();
+  else{
+    fCutX =  recParam->GetTrkCutX();
+    fCutY =  recParam->GetTrkCutY();
+    fCutZ =  recParam->GetTrkCutZ();
+    fMaxDist =  recParam->GetTrkCutR();
+    fCutAngle =  recParam->GetTrkCutAngle();
+    fCutAlphaMin =  recParam->GetTrkCutAlphaMin();
+    fCutAlphaMax =  recParam->GetTrkCutAlphaMax();
+    fCutNITS = recParam->GetTrkCutNITS();
+    fCutNTPC = recParam->GetTrkCutNTPC();
+  }
        
 }
 //
@@ -554,7 +555,7 @@ Double_t AliEMCALTracker::CheckPair
        //
        
        // TEMP
-       Bool_t isTrue = kFALSE;
+       //Bool_t isTrue = kFALSE;
 //     if (tr->GetSeedLabel() == cl->Label()) {
 //             isTrue = kTRUE;
 //     }
@@ -605,7 +606,7 @@ Double_t AliEMCALTracker::CheckPair
                Double_t r=0.;
                cout.setf(ios::fixed);
                cout.precision(5);
-               if (isTrue) cout << "Init : " << rt << ' ' << x << ' ' << y << ' ' << z << endl;
+               //if (isTrue) cout << "Init : " << rt << ' ' << x << ' ' << y << ' ' << z << endl;
                for (i = 0; i < fNPropSteps; i++) {
                        r = rt + (rc - rt) * ((Double_t)(i+1)/(Double_t)fNPropSteps);
                        if (!tr->PropagateTo(r, x0, rho)){
@@ -613,9 +614,9 @@ Double_t AliEMCALTracker::CheckPair
                          return distance;
                        }
                        tr->GetXYZ(pos);
-                       if (isTrue) cout << "Step : " << r << ' ' << x << ' ' << y << ' ' << z << endl;
+               //      if (isTrue) cout << "Step : " << r << ' ' << x << ' ' << y << ' ' << z << endl;
                }
-               if (isTrue) cout << "Clstr: " << rc << ' ' << cl->X() << ' ' << cl->Y() << ' ' << cl->Z() << endl;
+               //if (isTrue) cout << "Clstr: " << rc << ' ' << cl->X() << ' ' << cl->Y() << ' ' << cl->Z() << endl;
        }
        else {
                // when no steps are used, no correction makes sense