Coverity fixes.
authorcvetan <cvetan@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Oct 2010 13:43:34 +0000 (13:43 +0000)
committercvetan <cvetan@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Oct 2010 13:43:34 +0000 (13:43 +0000)
VZERO/AliVZERODataDCS.cxx
VZERO/AliVZERODigitizer.cxx
VZERO/AliVZEROQAChecker.cxx
VZERO/AliVZEROQADataMakerSim.cxx
VZERO/AliVZEROTrending.cxx
VZERO/AliVZEROTriggerSimulator.cxx
VZERO/AliVZEROv7.cxx

index b2f690c..6dd5277 100644 (file)
@@ -58,6 +58,7 @@ AliVZERODataDCS::AliVZERODataDCS():
                fDeadChannel[i] = kFALSE;
                fMeanHV[i]      = 100.0;
                fWidthHV[i]     = 0.0; 
+               fHv[i]          = NULL;
        }
 }
 
index 4945388..798911e 100644 (file)
@@ -401,6 +401,7 @@ void AliVZERODigitizer::DigitizeHits()
           Int_t nPhot = hit->Nphot();
           Int_t cell  = hit->Cell();                          
           Int_t pmt = Cell2Pmt(cell);
+          if (pmt < 0) continue;
           Int_t trackLabel = hit->GetTrack();
           for(Int_t l = 0; l < 3; ++l) {
             if (fLabels[pmt][l] < 0) {
index a65c104..a14f743 100644 (file)
@@ -192,6 +192,8 @@ void AliVZEROQAChecker::Init(const AliQAv1::DETECTORINDEX_t det)
   lowValue[AliQAv1::kFATAL]     = -1.0   ; 
   hiValue[AliQAv1::kFATAL]      = 0.0 ; 
   SetHiLo(hiValue, lowValue) ; 
+  delete [] hiValue;
+  delete [] lowValue;
 }
 
 //______________________________________________________________________________
index ac5fd06..95db649 100644 (file)
@@ -101,11 +101,8 @@ void AliVZEROQADataMakerSim::InitDigits()
   const Bool_t expert   = kTRUE ; 
   const Bool_t image    = kTRUE ; 
   
-  char TDCname[100];
-  char ADCname[100];
   TH1I *fhDigTDC[64]; 
   TH1I *fhDigADC[64]; 
-  char texte[100];
 
   // create Digits histograms in Digits subdir
   TH1I * h0 = new TH1I("hDigitMultiplicity", "Digits multiplicity distribution in VZERO;# of Digits;Entries", 100, 0, 99) ; 
@@ -114,13 +111,8 @@ void AliVZEROQADataMakerSim::InitDigits()
   
   for (Int_t i=0; i<64; i++)
     {
-       sprintf(TDCname, "hDigitTDC%d", i);
-       sprintf(texte,"Digit TDC in cell %d; TDC value;Entries",i);    
-       fhDigTDC[i] = new TH1I(TDCname,texte,300,0.,149.);
-       
-       sprintf(ADCname,"hDigitADC%d",i);
-       sprintf(texte,"Digit ADC in cell %d;ADC value;Entries",i);
-       fhDigADC[i]= new TH1I(ADCname,texte,1024,0.,1023.);
+       fhDigTDC[i] = new TH1I(Form("hDigitTDC%d", i),Form("Digit TDC in cell %d; TDC value;Entries",i),300,0.,149.);
+       fhDigADC[i]= new TH1I(Form("hDigitADC%d", i),Form("Digit ADC in cell %d;ADC value;Entries",i),1024,0.,1023.);
        
        Add2DigitsList(fhDigTDC[i],i+1, !expert, image);
        Add2DigitsList(fhDigADC[i],i+1+64, !expert, image);  
index fadd83c..a14644c 100644 (file)
@@ -35,6 +35,12 @@ AliVZEROTrending::AliVZEROTrending() : TH1(), fNEntries(0), fMultiGraphs(NULL)
 {\r
        // Default constructor\r
        for(int i=0; i<8;i++) fGraphs[i] = NULL;\r
+       for (int i = 0; i < kDataSize; i++) {\r
+               fTime[i] = 0;\r
+               for (int j = 0; j < 8; j++) {\r
+                 fData[j][i] = 0;\r
+               }\r
+       }\r
 }\r
 //_____________________________________________________________________________\r
 AliVZEROTrending::AliVZEROTrending(const char* name, const char* title) : TH1(), fNEntries(0), fMultiGraphs(NULL)\r
@@ -42,6 +48,12 @@ AliVZEROTrending::AliVZEROTrending(const char* name, const char* title) : TH1(),
        SetName(name);\r
        SetTitle(title);\r
        for(int i=0; i<8;i++) fGraphs[i] = NULL;\r
+       for (int i = 0; i < kDataSize; i++) {\r
+               fTime[i] = 0;\r
+               for (int j = 0; j < 8; j++) {\r
+                 fData[j][i] = 0;\r
+               }\r
+       }\r
 }\r
 //_____________________________________________________________________________\r
 AliVZEROTrending::AliVZEROTrending(const AliVZEROTrending &trend) : \r
index b9dbc87..9dc11c2 100644 (file)
@@ -160,6 +160,7 @@ void AliVZEROTriggerSimulator::Run() {
                        Int_t pmNumber   = digit->PMNumber();\r
                        Int_t board   = AliVZEROCalibData::GetBoardNumber(pmNumber);\r
                        Int_t channel = AliVZEROCalibData::GetFEEChannelNumber(pmNumber);\r
+                       if (board < 0 || channel < 0) continue;\r
                        \r
                        if(fTriggerData->GetEnableCharge(board,channel)) {\r
                                fCharges[pmNumber] = digit->ChargeADC(AliVZEROdigit::kNClocks/2);\r
index 47496c4..1e52a6f 100644 (file)
@@ -3173,18 +3173,17 @@ void AliVZEROv7::MakeBranch(Option_t *option)
 {
 // Creates new branches in the current Root Tree
     
-  char branchname[10];
-  sprintf(branchname,"%s",GetName());
+  TString branchname(Form("%s",GetName()));
   AliDebug(2,Form("fBufferSize = %d",fBufferSize));
   const char *cH = strstr(option,"H");
   if (fHits   && fLoader->TreeH() && cH) {
-    fLoader->TreeH()->Branch(branchname,&fHits, fBufferSize);
-    AliDebug(2,Form("Making Branch %s for hits",branchname));
+    fLoader->TreeH()->Branch(branchname.Data(),&fHits, fBufferSize);
+    AliDebug(2,Form("Making Branch %s for hits",branchname.Data()));
   }     
   const char *cD = strstr(option,"D");
   if (fDigits   && fLoader->TreeD() && cD) {
-    fLoader->TreeD()->Branch(branchname,&fDigits, fBufferSize);
-    AliDebug(2,Form("Making Branch %s for digits",branchname));
+    fLoader->TreeD()->Branch(branchname.Data(),&fDigits, fBufferSize);
+    AliDebug(2,Form("Making Branch %s for digits",branchname.Data()));
   }  
 }