Changes for bug #70680: AliROOT Coverity DELETE_ARRAY checker fix
authorhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 27 Jul 2010 09:27:40 +0000 (09:27 +0000)
committerhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 27 Jul 2010 09:27:40 +0000 (09:27 +0000)
38 files changed:
CORRFW/AliCFContainer.cxx
EMCAL/AliEMCALSDigitizer.cxx
EVE/EveDet/AliEveListAnalyserEditor.cxx
EVE/EveDet/AliEveTRDTrackList.cxx
EVE/EveDet/AliEveTRDTrackListEditor.cxx
HLT/trigger/AliHLTD0toKpi.cxx
HMPID/AliHMPIDReconHTA.cxx
ITS/AliITS.cxx
ITS/AliITSBeamTestDigSSD.cxx
ITS/AliITSOnlineSPDfoAnalyzer.cxx
ITS/AliITSgeomSSD.cxx
JETAN/AliEventShape.cxx
JETAN/AliUA1JetFinderV2.cxx
PHOS/AliPHOSAnalyze.cxx
PWG1/TPC/AliTPCtaskPID.cxx
PWG1/TPC/AliTPCtaskQA.cxx
PWG2/FEMTOSCOPY/AliFemto/AliFemtoEventReaderAOD.cxx
PWG2/FEMTOSCOPY/AliFemto/AliFemtoEventReaderESDChainKine.cxx
PWG2/SPECTRA/AliAnalysisTaskChargedHadronSpectra.cxx
PWG3/hfe/AliHFEcollection.cxx
STAT/AliTMinuitToolkit.cxx
STEER/AliVertexerTracks.cxx
TOF/AliTOFGeometry.cxx
TOF/AliTOFLvHvDataPoints.cxx
TPC/AliTPCCalibCE.cxx
TPC/AliTPCCalibTCF.cxx
TPC/AliTPCCalibViewer.cxx
TPC/AliTPCCalibViewerGUItime.cxx
TPC/AliTPCDigitizer.cxx
TPC/AliTPCMonitor.cxx
TPC/AliTPCcalibBase.cxx
TPC/AliTPCcalibCosmic.cxx
TPC/AliTPCcalibPID.cxx
TPC/AliTPCcalibTimeGain.cxx
TPC/AliTPCcalibV0.cxx
TRD/AliTRDPreprocessor.cxx
TTherminator/AliGenTherminator.cxx
TUHKMgen/AliGenUHKM.cxx

index 9467c82..24144c7 100644 (file)
@@ -463,7 +463,7 @@ void AliCFContainer::Print(const Option_t*) const {
     Double_t *binLimits = new Double_t[GetNBins(iVar)+1];
     GetBinLimits(iVar,binLimits);
     AliInfo(Form("variable %d \t -> %s : %d bins in [%f,%f]",iVar,GetVarTitle(iVar),GetNBins(iVar),binLimits[0],binLimits[GetNBins(iVar)]));
-    delete binLimits;
+    delete[] binLimits;
   }
   AliInfo("====================================================================================");
 }
index 09b9d35..24de52b 100644 (file)
@@ -465,7 +465,7 @@ void AliEMCALSDigitizer::PrintSDigits(Option_t * option)
        printf("%s",tempo); 
       }         
     }
-    delete tempo ;
+    delete [] tempo ;
     printf("\n** Sum %2.3f : %10.3f GeV/c **\n ", isum, Calibrate(isum));
   } else printf("\n");
 }
index 15494f2..f9d2602 100644 (file)
@@ -771,7 +771,7 @@ void AliEveListAnalyserEditor::HandleMacroPathSet()
         fteField->SetText(pathname);
 
         AddMacro(name);  
-        if (name != 0)  delete name;
+        if (name != 0)  delete [] name;
         name = 0;
       }
       // Different path
index 6a3d4e9..612ee3c 100644 (file)
@@ -594,7 +594,7 @@ Bool_t AliEveTRDTrackList::ApplyProcessMacros(const TList* selIterator, const TL
 
   if (procCmds != 0)  delete [] procCmds;
   procCmds = 0;
-  if (mProcType != 0)  delete mProcType;
+  if (mProcType != 0)  delete [] mProcType;
   mProcType = 0;
 
   if (selCmds != 0)  delete [] selCmds;
index 9b01fd2..18702fd 100644 (file)
@@ -685,7 +685,7 @@ void AliEveTRDTrackListEditor::HandleMacroPathSet()
         fteField->SetText(pathname);
 
         AddMacro(name);  
-        if (name != 0)  delete name;
+        if (name != 0)  delete [] name;
         name = 0;
       }
       // Different path
index 05be6bc..ecc9cbd 100644 (file)
@@ -96,7 +96,7 @@ AliAODVertex* AliHLTD0toKpi::ReconstructSecondaryVertex(TObjArray *trkArray, Dou
     id[0]=(UShort_t) t1->GetID();
     id[1]=(UShort_t) t2->GetID();
     vertexESD = (AliESDVertex*)vertexer->VertexForSelectedTracks(trkArray,id);
-    delete id;
+    delete [] id;
     delete vertexer; vertexer=NULL;
     
     if(!vertexESD) return vertexAOD;
index 9b0e223..949dd77 100644 (file)
@@ -860,7 +860,7 @@ Bool_t AliHMPIDReconHTA::UniformDistrib()
      chi2+= (iPhiBin[i] - theo)*(iPhiBin[i] - theo)/theo;
    }
    
-    delete iPhiBin;
+    delete [] iPhiBin;
     
     Double_t prob = TMath::Prob(chi2, nPhiBins-1);
     AliDebug(1,Form(" Probability for uniform distrib: %6f.3 %s",prob,(prob<0.05) ? "rejected" : "accepted"));
index 09d5d60..bf25ca0 100644 (file)
@@ -231,7 +231,7 @@ AliITS::~AliITS(){
     delete[] fIdSens;
     Int_t size   = 0;
     if (fDetTypeSim){
-      size = GetITSgeom()->GetIndexMax();
+      if (GetITSgeom()) size = GetITSgeom()->GetIndexMax();
       delete fDetTypeSim;
       fDetTypeSim = 0;
     }
index 35b6e48..52c83a1 100644 (file)
@@ -116,7 +116,7 @@ void AliITSBeamTestDigSSD::Exec(Option_t* /*opt*/) {
   }
   
   delete [] newdigits;
-  delete idig;
+  delete [] idig;
 }
 
   
index 3bb42c7..7582bea 100644 (file)
@@ -441,7 +441,7 @@ Bool_t AliITSOnlineSPDfoAnalyzer::IsExisting(TArrayI dacs,Int_t hs, Int_t chip)
   for(Int_t i=0; i<size; i++) entry[i] = dacs.At(i);
   Int_t checkbin = fNh[dacs.At(dacs.GetSize()-1)][hs][chip]->GetBin(entry,kFALSE); // kFALSE does not allocate another bin
   if(checkbin > -1) isOk = kTRUE;
-  delete entry;
+  delete [] entry;
   return isOk; 
 }
 //-----------------------------------------------------------
@@ -490,10 +490,10 @@ TArrayI AliITSOnlineSPDfoAnalyzer::GetCentralDACS(Int_t qualityflag, Int_t hs, I
   dacs.AddAt(qualityflag,fNdims);
   
   
-  delete mean;
-  delete goodbins;
-  delete bins;
-  delete val;
+  delete [] mean;
+  delete [] goodbins;
+  delete [] bins;
+  delete [] val;
   return dacs;
 }
 //-------------------------------------------------------
index e61cb49..026f5fa 100644 (file)
@@ -289,8 +289,8 @@ AliITSgeomSSD175::AliITSgeomSSD175() : AliITSgeomSSD(){
     leC[kNstrips] = -kDxyz[0];
     AliITSgeomSSD::ResetSSD(kDxyz,kangle,-kangle,
                                 kNstrips+1,leA,kNstrips+1,leC);
-    delete leA;
-    delete leC;
+    delete [] leA;
+    delete [] leC;
 }
 //________________________________________________________________________
 ostream &operator<<(ostream &os,AliITSgeomSSD175 &p){
@@ -444,8 +444,8 @@ AliITSgeomSSD75and275::AliITSgeomSSD75and275(Int_t npar,Float_t *par) :
     leC[kNstrips] = -kDxyz[0];
     AliITSgeomSSD::ResetSSD(par,kangleA,kangleC,
                                 kNstrips+1,leA,kNstrips+1,leC);
-    delete leA;
-    delete leC;
+    delete [] leA;
+    delete [] leC;
 }
 //________________________________________________________________________
 ostream &operator<<(ostream &os,AliITSgeomSSD75and275 &p){
index 8c8712d..09485dd 100644 (file)
@@ -186,9 +186,9 @@ TArrayD * AliEventShape::GetThrustParamMC(AliMCEvent* mcEvent, Int_t  nstudymin,
 
 
   delete evsh;
-  delete ptT;
-  delete pxT;
-  delete pyT;
+  delete [] ptT;
+  delete [] pxT;
+  delete [] pyT;
 
   return evsh;  
 }
index 16d3f9b..913cf7b 100644 (file)
@@ -279,13 +279,13 @@ void AliUA1JetFinderV2::FindJetsC()
   delete[] ncells;
   delete[] mult;
   //--- Added for jet reordering
-  delete etaJetOk;
-  delete phiJetOk;
-  delete etJetOk;
-  delete etsigJetOk;
-  delete etallJetOk;
-  delete ncellsJetOk;
-  delete multJetOk;
+  delete [] etaJetOk;
+  delete [] phiJetOk;
+  delete [] etJetOk;
+  delete [] etsigJetOk;
+  delete [] etallJetOk;
+  delete [] ncellsJetOk;
+  delete [] multJetOk;
   //--------------------------
 
 }
@@ -713,21 +713,21 @@ void AliUA1JetFinderV2::FindJets()
   delete etaCell2;
   delete phiCell2;
   delete flagCell2;
-  delete etaJet;
-  delete phiJet;
-  delete etJet;
-  delete etsigJet;
-  delete etallJet;
-  delete ncellsJet;
-  delete multJet;
+  delete [] etaJet;
+  delete [] phiJet;
+  delete [] etJet;
+  delete [] etsigJet;
+  delete [] etallJet;
+  delete [] ncellsJet;
+  delete [] multJet;
   //--- Added for jet reordering
-  delete etaJetOk;
-  delete phiJetOk;
-  delete etJetOk;
-  delete etsigJetOk;
-  delete etallJetOk;
-  delete ncellsJetOk;
-  delete multJetOk;
+  delete [] etaJetOk;
+  delete [] phiJetOk;
+  delete [] etJetOk;
+  delete [] etsigJetOk;
+  delete [] etallJetOk;
+  delete [] ncellsJetOk;
+  delete [] multJetOk;
   //--------------------------
   delete trackinjet;
   delete idxjets;
@@ -930,7 +930,7 @@ void AliUA1JetFinderV2::RunAlgoritm(Int_t nIn, Float_t* etCell, Float_t* const e
     }
 
   //delete
-  delete index;
+  delete[] index;
 
 }
 
@@ -1134,8 +1134,8 @@ void AliUA1JetFinderV2::RunAlgoritmC(Float_t etbgTotal, Double_t dEtTotal, Int_t
     }
   
   //delete
-  delete index;
-  delete idx;
+  delete [] index;
+  delete [] idx;
 
 }
 
index 100d511..51384ca 100644 (file)
@@ -636,7 +636,7 @@ void AliPHOSAnalyze::Ls(){
   mfile->Write();
   mfile->Close();
   delete mfile ;
-  delete nRecParticles;
+  delete [] nRecParticles;
 
 }
 
index 3e6f573..fd3080d 100644 (file)
@@ -406,5 +406,5 @@ void AliTPCtaskPID::BinLogX(TAxis *axis) {
     new_bins[i] = factor * new_bins[i-1];
   }
   axis->Set(bins, new_bins);
-  delete new_bins;
+  delete [] new_bins;
 }
index b4e5d0c..038d295 100644 (file)
@@ -331,7 +331,7 @@ void AliTPCtaskQA::BinLogX(TAxis *axis) {
     new_bins[i] = factor * new_bins[i-1];
   }
   axis->Set(bins, new_bins);
-  delete new_bins;
+  delete [] new_bins;
 }
 
 
index 808ae08..2a81231 100644 (file)
@@ -520,7 +520,7 @@ void AliFemtoEventReaderAOD::CopyAODtoFemtoEvent(AliFemtoEvent *tEvent)
 
   tEvent->SetNumberOfTracks(realnofTracks);//setting number of track which we read in event    
 
-  if (mcP) delete motherids;
+  if (mcP) delete [] motherids;
 
   cout<<"end of reading nt "<<nofTracks<<" real number "<<realnofTracks<<endl;
 }
index 29314a2..4f339b6 100644 (file)
@@ -529,7 +529,7 @@ AliFemtoEvent* AliFemtoEventReaderESDChainKine::ReturnHbtEvent()
                
     }
 
-  delete motherids;
+  delete [] motherids;
   
   hbtEvent->SetNumberOfTracks(realnofTracks);//setting number of track which we read in event  
   fCurEvent++; 
index 16ee981..e8c0df1 100644 (file)
@@ -686,7 +686,7 @@ void AliAnalysisTaskChargedHadronSpectra::BinLogX(const TH1 *h) {
    newBins[i] = factor * newBins[i-1];
   }
   axis->Set(bins, newBins);
-  delete newBins;
+  delete [] newBins;
   
 }
 
index bc33ad0..012cce6 100644 (file)
@@ -469,7 +469,7 @@ Bool_t AliHFEcollection::BinLogAxis(const char* name, Int_t dim){
     newBins[i] = factor * newBins[i-1];
   }
   axis->Set(bins, newBins);
-  delete newBins;
+  delete [] newBins;
 
   return kTRUE;
 
index 6feb618..d79a24e 100644 (file)
@@ -385,8 +385,8 @@ void AliTMinuitToolkit::Fit() {
    if (fMaxCalls != 500 || fPrecision != 1) minuit->ExecuteCommand(fFitAlgorithm, argList, 2);
    fUseRobust = true;
    
-   delete sortList; 
-   delete indexList;    
+   delete [] sortList; 
+   delete [] indexList;    
   }
   
   // fill parameter vector
index 5dd33ab..b8cf3d7 100644 (file)
@@ -212,7 +212,7 @@ AliESDVertex* AliVertexerTracks::FindPrimaryVertex(const AliVEvent *vEvent)
   FindPrimaryVertex(&trkArrayOrig,idOrig);
 
   if(fMode==0) trkArrayOrig.Delete();
-  delete idOrig; idOrig=NULL;
+  delete[] idOrig; idOrig=NULL;
 
   if(f) {
     f->Close(); delete f; f = NULL;
@@ -265,7 +265,7 @@ AliESDVertex* AliVertexerTracks::FindPrimaryVertex(const TObjArray *trkArrayOrig
     // fill fTrkArraySel, for VertexFinder()
     fIdSel = new UShort_t[nTrksOrig];
     PrepareTracks(*trkArrayOrig,idOrig,0);
-    if(fIdSel) { delete fIdSel; fIdSel=NULL; }
+    if(fIdSel) { delete[] fIdSel; fIdSel=NULL; }
     Double_t cutsave = fDCAcut;  
     fDCAcut = fDCAcutIter0;
     // vertex finder
@@ -832,7 +832,7 @@ AliESDVertex* AliVertexerTracks::RemoveTracksFromVertex(AliESDVertex *inVtx,
     }
   }
   outVtx->SetIndices(nIndices,outindices);
-  if (outindices) delete outindices;
+  if (outindices) delete [] outindices;
 
   /*
     printf("Vertex before removing tracks:");
@@ -1203,8 +1203,8 @@ AliESDVertex AliVertexerTracks::TrackletVertexFinder(AliStrLine **lines, const I
   }
   AliESDVertex theVert(initPos,covmatrix,99999.,knacc);
   theVert.SetDispersion(sigma);
-  delete vectP0;
-  delete vectP1;
+  delete [] vectP0;
+  delete [] vectP1;
   return theVert;
 }
 
@@ -1706,7 +1706,7 @@ AliESDVertex* AliVertexerTracks::VertexForSelectedTracks(const TObjArray *trkArr
   }
 
   // clean up
-  if (indices) {delete indices; indices=NULL;}
+  if (indices) {delete [] indices; indices=NULL;}
   delete fIdSel; fIdSel=NULL;
   fTrkArraySel.Delete();
   
@@ -1734,7 +1734,7 @@ AliESDVertex* AliVertexerTracks::VertexForSelectedESDTracks(TObjArray *trkArray,
     
   VertexForSelectedTracks(trkArray,id,optUseFitter,optPropagate,optUseDiamondConstraint);
 
-  delete id; id=NULL;
+  delete [] id; id=NULL;
 
   return fCurrentVertex;
 }
index 308cb67..e558af4 100644 (file)
@@ -1445,7 +1445,7 @@ void AliTOFGeometry::DetToSectorRF(Int_t vol[5], Double_t **coord)
 
   for (Int_t jj=0; jj<4; jj++) padToSector.MasterToLocal(&cornerPad[jj][0], &coord[jj][0]);
 
-  delete cornerPad;
+  delete [] cornerPad;
 
   //sectorToPad.LocalToMaster(cornerPad, coord);
 
index bc30579..507749d 100644 (file)
@@ -302,7 +302,7 @@ Bool_t AliTOFLvHvDataPoints::MergeMaps() {
 
   }
 
-  delete controller;
+  delete [] controller;
 
   for (Int_t ii=0; ii<fNumberOfHVandLVmaps; ii++)
     AliDebug(1,Form(" fMap[%d]->GetTime() = %d ",ii,fMap[ii]->GetTime()));
index 3b644b2..838afcd 100644 (file)
@@ -1727,9 +1727,9 @@ TGraph *AliTPCCalibCE::MakeGraphTimeCE(Int_t sector, Int_t xVariable, Int_t fitT
 
 
   if ( xVariable == 2 ) delete xVar;
-  delete x;
-  delete y;
-  delete sortIndex;
+  delete [] x;
+  delete [] y;
+  delete [] sortIndex;
   return gr;
 }
 //_____________________________________________________________________
index 5511e7e..d421a7b 100644 (file)
@@ -1626,8 +1626,8 @@ TNtuple *AliTPCCalibTCF::ApplyTCFilter(TH1F *hisIn, Double_t *coefZ, Double_t *c
 
   signalIn->~Double_t();
   signalOut->~Double_t();
-  delete signalIn;
-  delete signalOut;
+  delete [] signalIn;
+  delete [] signalOut;
 
   return pulseTuple;
 
index 13e5dfd..c092ab6 100644 (file)
@@ -305,8 +305,8 @@ const char* AliTPCCalibViewer::AddAbbreviations(const Char_t *c, Bool_t printDra
    str.ReplaceAll(removeString, "");
   
    if (printDrawCommand) std::cout << "The string looks now like: " << str.Data() << std::endl;
-   delete varSort;
-   delete normSort;
+   delete [] varSort;
+   delete [] normSort;
    return str.Data();
 }
 
index 8314d4e..5583ddf 100644 (file)
@@ -964,7 +964,7 @@ void AliTPCCalibViewerGUItime::DoDumpRuns()
     if (run!=prevRun) std::cout << Form("%d",run) << std::endl;
     prevRun=run;
   }
-  delete sortIndex;
+  delete [] sortIndex;
 }
 //______________________________________________________________________________
 void AliTPCCalibViewerGUItime::DoParLimitChange()
index b8c4677..ed62f30 100644 (file)
@@ -535,6 +535,6 @@ void AliTPCDigitizer::ExecSave(Option_t* option)
     
   delete digrow;     
   for (Int_t i1=0;i1<nInputs; i1++) delete digarr[i1];
-  delete []masks;
-  delete digarr;  
+  delete [] masks;
+  delete [] digarr;  
 }
index 5bdfea0..b5b1329 100755 (executable)
@@ -1874,7 +1874,7 @@ void AliTPCMonitor::ExecTransform()
   htransmag->Draw();
   ctrans2->Update();
   delete four;
-  delete data;
+  delete [] data;
 }
 
 //__________________________________________________________________
index 79fffbb..aff72e6 100644 (file)
@@ -383,7 +383,7 @@ void AliTPCcalibBase::BinLogX(THnSparse *h, Int_t axisDim) {
    new_bins[i] = factor * new_bins[i-1];
   }
   axis->Set(bins, new_bins);
-  delete new_bins;
+  delete [] new_bins;
 
 }
 void AliTPCcalibBase::BinLogX(TH1 *h) {
@@ -404,6 +404,6 @@ void AliTPCcalibBase::BinLogX(TH1 *h) {
    new_bins[i] = factor * new_bins[i-1];
   }
   axis->Set(bins, new_bins);
-  delete new_bins;
+  delete [] new_bins;
 
 }
index 3aaa6dd..aea04cf 100644 (file)
@@ -884,7 +884,7 @@ void AliTPCcalibCosmic::BinLogX(THnSparse *const h, Int_t axisDim) {
    newBins[i] = factor * newBins[i-1];
   }
   axis->Set(bins, newBins);
-  delete newBins;
+  delete [] newBins;
 
 }
 
@@ -907,7 +907,7 @@ void AliTPCcalibCosmic::BinLogX(TH1 *const h) {
    newBins[i] = factor * newBins[i-1];
   }
   axis->Set(bins, newBins);
-  delete newBins;
+  delete [] newBins;
   
 }
 
index bdf45b9..ff29781 100644 (file)
@@ -499,7 +499,7 @@ void AliTPCcalibPID::BinLogX(THnSparse *h, Int_t axisDim) {
    newBins[i] = factor * newBins[i-1];
   }
   axis->Set(bins, newBins);
-  delete newBins;
+  delete [] newBins;
   
 }
 
index 2a80899..11093d4 100644 (file)
@@ -583,7 +583,7 @@ void AliTPCcalibTimeGain::BinLogX(THnSparse *h, Int_t axisDim) {
    newBins[i] = factor * newBins[i-1];
   }
   axis->Set(bins, newBins);
-  delete newBins;
+  delete[] newBins;
   
 }
 
@@ -606,7 +606,7 @@ void AliTPCcalibTimeGain::BinLogX(TH1 *h) {
    newBins[i] = factor * newBins[i-1];
   }
   axis->Set(bins, newBins);
-  delete newBins;
+  delete[] newBins;
   
 }
 
index 3840657..96c0ca4 100644 (file)
@@ -778,7 +778,7 @@ void AliTPCcalibV0::BinLogX(TH2F *h) {
      new_bins[i] = factor * new_bins[i-1];
    }
    axis->Set(bins, new_bins);
-   delete new_bins;
+   delete [] new_bins;
    
 }
 
index be94fad..8170ffe 100644 (file)
@@ -303,8 +303,8 @@ Bool_t AliTRDPreprocessor::ProcessDCS(TMap *dcsAliasMap)
   }
   Log ("*********** End of DCS **********");
   
-  delete results;
-  delete nGraph;
+  delete [] results;
+  delete [] nGraph;
 
   return error;
 
index b75b248..987a1dd 100644 (file)
@@ -296,7 +296,7 @@ void AliGenTherminator::Generate()
   AddHeader(header);
   fCollisionGeometry = (AliGenHijingEventHeader*)  header;
 
-  delete idsOnStack;
+  delete [] idsOnStack;
 
   //  gAlice->SetGenEventHeader(header); 
 }
index e5c829a..34bbdf8 100755 (executable)
@@ -467,7 +467,7 @@ void AliGenUHKM::Generate()
   AddHeader(header);
   fCollisionGeometry = (AliGenHijingEventHeader*)  header;
 
-  delete idsOnStack;
+  delete [] idsOnStack;
 
 }