From fd20854c5a878b1a5df7c834985ab6e7b89185ba Mon Sep 17 00:00:00 2001 From: dainese Date: Mon, 31 Jan 2011 23:59:58 +0000 Subject: [PATCH] Coverity --- PWG3/vertexingHF/AliAnalysisTaskSEVertexingHF.cxx | 1 + PWG3/vertexingHF/AliAnalysisVertexingHF.cxx | 11 ++++++----- PWG3/vertexingHF/AliRDHFCuts.cxx | 5 ++--- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/PWG3/vertexingHF/AliAnalysisTaskSEVertexingHF.cxx b/PWG3/vertexingHF/AliAnalysisTaskSEVertexingHF.cxx index 56d928d9faf..67141618d55 100644 --- a/PWG3/vertexingHF/AliAnalysisTaskSEVertexingHF.cxx +++ b/PWG3/vertexingHF/AliAnalysisTaskSEVertexingHF.cxx @@ -238,6 +238,7 @@ void AliAnalysisTaskSEVertexingHF::UserExec(Option_t */*option*/) Float_t dummy; AliESDUtils::GetCorrV0(eventE,dummy,vChCorr); fHFUtilInfo->SetVZERO( vChCorr ); + delete [] vChCorr; } //-------------------------------------------------------------- diff --git a/PWG3/vertexingHF/AliAnalysisVertexingHF.cxx b/PWG3/vertexingHF/AliAnalysisVertexingHF.cxx index 233031e8632..1db45fcb0eb 100644 --- a/PWG3/vertexingHF/AliAnalysisVertexingHF.cxx +++ b/PWG3/vertexingHF/AliAnalysisVertexingHF.cxx @@ -1371,12 +1371,8 @@ AliAODRecoCascadeHF* AliAnalysisVertexingHF::MakeCascade( } // select Cascades - Bool_t okLcksp=0, okLcLpi=0; if(fCascades && fInputAOD){ okCascades = (Bool_t)fCutsLctoV0->IsSelected(tmpCascade,AliRDHFCuts::kCandidate); - if(okCascades==1) okLcksp=1; - if(okCascades==2) okLcLpi=1; - if(okCascades==3) { okLcksp=1; okLcLpi=1;} } else { AliDebug(2,Form("The cascade is contructed from ESDs, no cuts are applied")); okCascades=kTRUE; }// no cuts implemented from ESDs tmpCascade->GetSecondaryVtx()->RemoveDaughters(); @@ -2220,7 +2216,10 @@ AliAODv0* AliAnalysisVertexingHF::TransformESDv0toAODv0(AliESDv0 *esdV0, TObjArr esdV0->PxPyPz(pxpypz); Double_t cv[21]; for(int i=0; i<21; i++) cv[i]=0; AliNeutralTrackParam *trackesdV0 = new AliNeutralTrackParam(xyz,pxpypz,cv,0); - if(!trackesdV0) return 0; + if(!trackesdV0) { + delete vertexV0; + return 0; + } Double_t d0z0[2],covd0z0[3]; AliAODVertex *primVertexAOD = PrimaryVertex(); trackesdV0->PropagateToDCA(primVertexAOD,fBzkG,kVeryBig,d0z0,covd0z0); @@ -2231,6 +2230,8 @@ AliAODv0* AliAnalysisVertexingHF::TransformESDv0toAODv0(AliESDv0 *esdV0, TObjArr AliExternalTrackParam *negV0track = (AliExternalTrackParam*)twoTrackArrayV0->UncheckedAt(1); if( !posV0track || !negV0track) { if(trackesdV0) {delete trackesdV0; trackesdV0=NULL;} + delete vertexV0; + delete primVertexAOD; return 0; } posV0track->PropagateToDCA(primVertexAOD,fBzkG,kVeryBig,d0z0,covd0z0); diff --git a/PWG3/vertexingHF/AliRDHFCuts.cxx b/PWG3/vertexingHF/AliRDHFCuts.cxx index 68a670c8d26..0278c7ce2db 100644 --- a/PWG3/vertexingHF/AliRDHFCuts.cxx +++ b/PWG3/vertexingHF/AliRDHFCuts.cxx @@ -235,8 +235,7 @@ Bool_t AliRDHFCuts::IsEventSelected(AliVEvent *event) { fWhyRejection=3; return kFALSE; }else{ - Float_t centvalue=GetCentrality((AliAODEvent*)event); - if (centvalue<0.){ + Float_t centvalue=GetCentrality((AliAODEvent*)event); if (centvalue<0.){ if (fWhyRejection==3) return kFALSE; else return kTRUE; } @@ -271,7 +270,7 @@ Bool_t AliRDHFCuts::AreDaughtersSelected(AliAODRecoDecayHF *d) const { for(Int_t idg=0; idgGetDaughter(idg); - if(!dgTrack) retval = kFALSE; + if(!dgTrack) {retval = kFALSE; continue;} //printf("charge %d\n",dgTrack->Charge()); if(dgTrack->Charge()==0) continue; // it's not a track, but a V0 -- 2.43.0