Coverity stuff once more ...
authorcblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Oct 2010 15:20:17 +0000 (15:20 +0000)
committercblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Oct 2010 15:20:17 +0000 (15:20 +0000)
16 files changed:
TRD/AliTRDCalibChamberStatus.cxx
TRD/AliTRDCalibTask.cxx
TRD/AliTRDCalibViewer.cxx
TRD/AliTRDCalibraFit.cxx
TRD/AliTRDCalibraVdriftLinearFit.cxx
TRD/AliTRDalignment.cxx
TRD/AliTRDarrayADC.cxx
TRD/AliTRDarrayDictionary.cxx
TRD/AliTRDclusterizer.cxx
TRD/AliTRDdigitsManager.cxx
TRD/Cal/AliTRDCalDet.cxx
TRD/Cal/AliTRDCalMonitoring.cxx
TRD/Cal/AliTRDCalPad.cxx
TRD/qaAnalysis/AliAnalysisTaskTRDmon.cxx
TRD/qaAnalysis/AliTRDqaJPsi.cxx
TRD/qaGui/AliTRDqaGuiBlackSM.cxx

index a91ef55..3bb3dc8 100644 (file)
@@ -280,6 +280,7 @@ void AliTRDCalibChamberStatus::ProcessEvent(AliRawReader * rawReader, Int_t neve
   rawStream->SetSharedPadReadout(kFALSE);
 
   AliTRDdigitsManager *digitsManager = new AliTRDdigitsManager(kTRUE);
+  if (!digitsManager) return;
   digitsManager->CreateArrays();
   
   Int_t det    = 0;
@@ -331,8 +332,8 @@ void AliTRDCalibChamberStatus::ProcessEvent(AliRawReader * rawReader, Int_t neve
 
   if(notEmpty) fCounterEventNotEmpty++;
 
-  if(digitsManager) delete digitsManager;
-  if(rawStream) delete rawStream;
+  delete digitsManager;
+  delete rawStream;
    
 }//_____________________________________________________________________
 void AliTRDCalibChamberStatus::ProcessEvent3(AliRawReader * rawReader, Int_t nevents_physics)
@@ -402,7 +403,7 @@ void AliTRDCalibChamberStatus::ProcessEvent3(AliRawReader * rawReader, Int_t nev
 
   if(notEmpty) fCounterEventNotEmpty++;
 
-  if(digitsManager) delete digitsManager;
+  delete digitsManager;
   delete rawStream;
    
 }
index 79660af..aadec5e 100644 (file)
@@ -934,7 +934,9 @@ void AliTRDCalibTask::Plot()
          first += 1;
        }
        else {
-         histolinearfitsum ->Add(linearfit->GetLinearFitterHisto(det));
+          if (histolinearfitsum) {
+           histolinearfitsum->Add(linearfit->GetLinearFitterHisto(det));
+         }
        }
       }
     }
index 17f13a4..c1eba0b 100644 (file)
@@ -360,7 +360,7 @@ void AliTRDCalibViewer::GetLayerSectorStack(TString trdString, Int_t& layerNo, I
   // encoded with the following format:
   // Layer%dSector%dStack%d
 
-  sscanf(trdString.Data(), "Layer%dSector%dStack%d", &layerNo, &sectorNo, &stackNo);
+  sscanf(trdString.Data(), "Layer%1dSector%02dStack%1d", &layerNo, &sectorNo, &stackNo);
 
   return;
 }
@@ -636,7 +636,7 @@ Int_t AliTRDCalibViewer::EasyDraw(const char* drawCommand, Int_t chamber, const
     Int_t stackNo = (chamber%30)/6;
     Int_t layerNo = (chamber%30)%6;
     char sectorChr[22];
-    sprintf(sectorChr, "Layer%iSector%iStack%i", layerNo, superModuleNo, stackNo);
+    snprintf(sectorChr,22, "Layer%iSector%iStack%i", layerNo, superModuleNo, stackNo);
     return EasyDraw(drawCommand, sectorChr, cuts, drawOptions, writeDrawCommand);
   }
   Error("EasyDraw","The TRD contains only chamber from 0 to 539");
@@ -659,7 +659,7 @@ Int_t AliTRDCalibViewer::EasyDraw1D(const char* drawCommand, Int_t chamber, cons
     Int_t stackNo = (chamber%30)/6;
     Int_t layerNo = (chamber%30)%6;
     char sectorChr[22];
-    sprintf(sectorChr, "Layer%iSector%iStack%i", layerNo, superModuleNo, stackNo);
+    snprintf(sectorChr,22, "Layer%iSector%iStack%i", layerNo, superModuleNo, stackNo);
     return EasyDraw1D(drawCommand, sectorChr, cuts, drawOptions, writeDrawCommand);
   }
   Error("EasyDraw1D","The TRD contains only chambers from 0 to 539");
index aa83e67..81f7543 100644 (file)
@@ -4886,8 +4886,8 @@ void AliTRDCalibraFit::FitLagrangePoly(TH1* projPH)
     pente->Draw();
   }
   else {
-    if(pentea) delete pentea;
-    if(pente) delete pente;
+    delete pentea;
+    delete pente;
     if(polynome) delete polynome;
     if(polynomea) delete polynomea;
     if(polynomeb) delete polynomeb;
index 80ce619..4a9c859 100644 (file)
@@ -351,8 +351,8 @@ void AliTRDCalibraVdriftLinearFit::FillPEArray()
       
     }// if something
 
-    delete [] arrayI;
-
   }
+
+  delete [] arrayI;
    
 }
index b2bc442..c341f2f 100644 (file)
@@ -178,6 +178,10 @@ AliTRDalignment::AliTRDalignment(const AliTRDalignment& source)
     fSurveyY0[j][k][l] = source.fSurveyY0[j][k][l];
     fSurveyZ0[j][k][l] = source.fSurveyZ0[j][k][l];
   }
+  for (int i=0; i<1000; i++) {
+    fIbuffer[i] = 0;
+    fDbuffer[i] = 0.0;
+  }
 
 }
 
index 4116907..7f28103 100644 (file)
@@ -418,8 +418,15 @@ void AliTRDarrayADC::Compress()
           buffer=0;
         } 
 
+    }
+
+  if (longz)
+    {
       delete [] longz;
       longz=0;
+    }
+  if (longm)
+    {
       delete [] longm;
       longm=0;
 
index d76dcad..d239fa3 100644 (file)
@@ -375,8 +375,11 @@ void AliTRDarrayDictionary::Expand()
 
        }
 
-      delete [] longArr;
-      
+    }
+
+  if (longArr)
+    {
+      delete [] longArr; 
     }
 
 }
index 5c4a8b5..de142a4 100644 (file)
@@ -111,7 +111,9 @@ AliTRDclusterizer::AliTRDclusterizer(const AliTRDReconstructor *const rec)
 }
 
 //_____________________________________________________________________________
-AliTRDclusterizer::AliTRDclusterizer(const Text_t *name, const Text_t *title, const AliTRDReconstructor *const rec)
+AliTRDclusterizer::AliTRDclusterizer(const Text_t *name
+                                   , const Text_t *title
+                                   , const AliTRDReconstructor *const rec)
   :TNamed(name,title)
   ,fReconstructor(rec)
   ,fRunLoader(NULL)
@@ -153,11 +155,6 @@ AliTRDclusterizer::AliTRDclusterizer(const Text_t *name, const Text_t *title, co
   SetBit(kLabels, kTRUE);
   SetBit(knewDM, kFALSE);
 
-  AliTRDcalibDB *trd = 0x0;
-  if (!(trd = AliTRDcalibDB::Instance())) {
-    AliFatal("Could not get calibration object");
-  }
-
   fDigitsManager->CreateArrays();
 
   fRawVersion = AliTRDfeeParam::Instance()->GetRAWversion();
index cb38cf3..048ee25 100644 (file)
@@ -878,7 +878,7 @@ Bool_t AliTRDdigitsManager::LoadArrayDict()
 
       // Get the branch
       Char_t branchname[15];
-      sprintf(branchname,"TRDdictionary%d",iDict);
+      snprintf(branchname,15,"TRDdictionary%d",iDict);
       TBranch *branch = fTree->GetBranch(branchname);
       if (!branch) 
         {
@@ -1013,7 +1013,7 @@ Bool_t AliTRDdigitsManager::StoreArrayDict()
 
        // Get the branch
        Char_t branchname[15];
-       sprintf(branchname,"TRDdictionary%d",iDict);
+       snprintf(branchname,15,"TRDdictionary%d",iDict);
        TBranch *branch = fTree->GetBranch(branchname);
        if (!branch) 
          {
index 2b2bce1..082181e 100644 (file)
@@ -279,7 +279,7 @@ TH1F * AliTRDCalDet::MakeHisto1DAsFunctionOfDet(Float_t min, Float_t max,Int_t t
   }
  
   char  name[1000];
-  sprintf(name,"%s CalDet as function of det",GetTitle());
+  snprintf(name,1000,"%s CalDet as function of det",GetTitle());
   TH1F * his = new TH1F(name,name,kNdet, 0, kNdet);
   for(Int_t det = 0; det< kNdet; det++){
     his->Fill(det+0.5,GetValue(det));
@@ -334,7 +334,7 @@ TH2F *AliTRDCalDet::MakeHisto2DCh(Int_t ch, Float_t min, Float_t max, Int_t type
   Double_t posglobal[3] = {0.0,0.0,0.0};
   
   char  name[1000];
-  sprintf(name,"%s CalDet 2D ch %d",GetTitle(),ch);
+  snprintf(name,1000,"%s CalDet 2D ch %d",GetTitle(),ch);
   TH2F * his = new TH2F(name, name, 400,-400.0,400.0,400,-400.0,400.0);
 
   // Where we begin
@@ -410,7 +410,7 @@ TH2F *AliTRDCalDet::MakeHisto2DSmPl(Int_t sm, Int_t pl, Float_t min, Float_t max
   Double_t col0    = padPlane0->GetCol0();
 
   char  name[1000];
-  sprintf(name,"%s CalDet 2D sm %d and pl %d",GetTitle(),sm,pl);
+  snprintf(name,1000,"%s CalDet 2D sm %d and pl %d",GetTitle(),sm,pl);
   TH2F * his = new TH2F( name, name, 5,  -TMath::Abs(row0),  TMath::Abs(row0)
                                    , 4,-2*TMath::Abs(col0),2*TMath::Abs(col0));
 
index 62c1697..9d48a37 100644 (file)
@@ -81,5 +81,9 @@ AliTRDCalMonitoring::AliTRDCalMonitoring(const Text_t *name, const Text_t *title
     fLVVoltage[i] = 0;
     fLVCurrent[i] = 0;
   }
+  for (Int_t i=0; i<6700; ++i) {
+    fADCTresholds[i] = 0;
+  }
+
 }
 
index 65ff0f0..277ba62 100644 (file)
@@ -450,7 +450,7 @@ TH2F *AliTRDCalPad::MakeHisto2DSmPl(Int_t sm, Int_t pl, const AliTRDCalDet *calD
   Double_t col0    = padPlane0->GetCol0();
 
   char  name[1000];
-  sprintf(name,"%s Pad 2D sm %d pl %d",GetTitle(),sm,pl);
+  snprintf(name,1000,"%s Pad 2D sm %d pl %d",GetTitle(),sm,pl);
   TH2F * his = new TH2F( name, name, 76,-TMath::Abs(row0),TMath::Abs(row0),144,-TMath::Abs(col0),TMath::Abs(col0));
 
   // Where we begin
@@ -550,7 +550,7 @@ TH2F *AliTRDCalPad::MakeHisto2DCh(Int_t ch, const AliTRDCalDet *calDet, Int_t ty
   Double_t posglobal[3] = {0.0,0.0,0.0};
   
   char  name[1000];
-  sprintf(name,"%s Pad 2D ch %d",GetTitle(),ch);
+  snprintf(name,1000,"%s Pad 2D ch %d",GetTitle(),ch);
   TH2F * his = new TH2F( name, name, 400,-400.0,400.0,400,-400.0,400.0);
 
   // Where we begin
index 2f03df1..a216190 100644 (file)
@@ -626,7 +626,6 @@ void AliAnalysisTaskTRDmon::Exec(Option_t *){
       for(Int_t itl = 0; itl < 6; itl++){ // TRD layers
        AliTRDseedV1 * tracklet = fTRDtrack->GetTracklet(itl);
        if(!tracklet || !tracklet->IsOK()) continue;
-       if(!tracklet) continue;
        nTracklets++;
        Int_t nclsTracklet = 0;
        for(Int_t itb = 0; itb < AliTRDseedV1::kNtb; itb++){ // timebins
index f60049f..baef274 100644 (file)
@@ -68,6 +68,10 @@ AliTRDqaJPsi:: AliTRDqaJPsi(const AliTRDqaJPsi & /*trd*/)
   // Copy constructor
   //
 
+  for (Int_t i = 0; i < 1000; i++) {
+    fSM[i] = 0;
+  }
+
 }
 
 //______________________________________________________________________________
index bed624d..768667c 100644 (file)
@@ -70,7 +70,8 @@ AliTRDqaGuiBlackSM::AliTRDqaGuiBlackSM()
     fRangeNoise[i] = 0.0;
   }
   for (Int_t i = 0; i < 30; i++) {
-    fHistList[i] = 0x0;
+    fHistList[i]   = 0x0;
+    fCanvasList[i] = 0x0;
   }
 
   strncpy(fFileName,"",256);