From a828d1359589312808ae2ea366a3551be7ca22b1 Mon Sep 17 00:00:00 2001 From: dainese Date: Tue, 29 Mar 2011 11:45:26 +0000 Subject: [PATCH] Coverity --- PWG3/vertexingHF/AliAnalysisTaskSEHFQA.cxx | 10 ++++++++-- PWG3/vertexingHF/AliAnalysisTaskSEImproveITS.cxx | 1 + PWG3/vertexingHF/AliCFVertexingHFCascade.cxx | 4 ++++ 3 files changed, 13 insertions(+), 2 deletions(-) diff --git a/PWG3/vertexingHF/AliAnalysisTaskSEHFQA.cxx b/PWG3/vertexingHF/AliAnalysisTaskSEHFQA.cxx index 3ac2f509150..2d97cd6f872 100644 --- a/PWG3/vertexingHF/AliAnalysisTaskSEHFQA.cxx +++ b/PWG3/vertexingHF/AliAnalysisTaskSEHFQA.cxx @@ -602,6 +602,10 @@ void AliAnalysisTaskSEHFQA::UserExec(Option_t */*option*/) break; } } + + + if(!aod) {delete [] pdgdaughters;return;} + Bool_t isSimpleMode=fSimpleMode; if(!arrayProng) { AliInfo("Branch not found! The output will contain only trak related histograms\n"); @@ -630,10 +634,12 @@ void AliAnalysisTaskSEHFQA::UserExec(Option_t */*option*/) return; } } - if(!aod) {delete [] pdgdaughters;return;} // fix for temporary bug in ESDfilter // the AODs with null vertex pointer didn't pass the PhysSel - if(!aod->GetPrimaryVertex() || TMath::Abs(aod->GetMagneticField())<0.001) return; + if(!aod->GetPrimaryVertex() || TMath::Abs(aod->GetMagneticField())<0.001) { + delete [] pdgdaughters; + return; + } // count event fNEntries->Fill(0); diff --git a/PWG3/vertexingHF/AliAnalysisTaskSEImproveITS.cxx b/PWG3/vertexingHF/AliAnalysisTaskSEImproveITS.cxx index b23ee923215..f0ad603c2b4 100644 --- a/PWG3/vertexingHF/AliAnalysisTaskSEImproveITS.cxx +++ b/PWG3/vertexingHF/AliAnalysisTaskSEImproveITS.cxx @@ -196,6 +196,7 @@ void AliAnalysisTaskSEImproveITS::UserExec(Option_t*) { // The event loop // AliAODEvent *ev=dynamic_cast(InputEvent()); + if(!ev) return; Double_t bz=ev->GetMagneticField(); // Smear all tracks diff --git a/PWG3/vertexingHF/AliCFVertexingHFCascade.cxx b/PWG3/vertexingHF/AliCFVertexingHFCascade.cxx index 30240ea07af..b60690a2f3d 100644 --- a/PWG3/vertexingHF/AliCFVertexingHFCascade.cxx +++ b/PWG3/vertexingHF/AliCFVertexingHFCascade.cxx @@ -136,6 +136,8 @@ Bool_t AliCFVertexingHFCascade::GetGeneratedValuesFromMCParticle(Double_t* vecto AliAODMCParticle* mcPartDaughterD0 = dynamic_cast(fmcArray->At(daughter0ds)); AliAODMCParticle* mcPartDaughterPis = dynamic_cast(fmcArray->At(daughter1ds)); + if(!mcPartDaughterD0 || !mcPartDaughterPis) return kFALSE; + Double_t vtx1[3] = {0,0,0}; // primary vertex Double_t vtx2daughter0[3] = {0,0,0}; // secondary vertex from daughter 0 Double_t vtx2daughter1[3] = {0,0,0}; // secondary vertex from daughter 1 @@ -148,6 +150,8 @@ Bool_t AliCFVertexingHFCascade::GetGeneratedValuesFromMCParticle(Double_t* vecto AliAODMCParticle* mcPartDaughter0 = dynamic_cast(fmcArray->At(daughter0)); //D0 AliAODMCParticle* mcPartDaughter1 = dynamic_cast(fmcArray->At(daughter1)); //pis + if(!mcPartDaughter0 || !mcPartDaughter1) return kFALSE; + // getting vertex from daughters mcPartDaughter0->XvYvZv(vtx2daughter0); // cm mcPartDaughter1->XvYvZv(vtx2daughter1); //cm -- 2.43.0