From 2db138e0b06ab7a61b65cee6fa15db2b5e7c7916 Mon Sep 17 00:00:00 2001 From: fprino Date: Mon, 17 Feb 2014 14:08:08 +0100 Subject: [PATCH] Fix for coverity defects --- PWGHF/vertexingHF/AliAnalysisTaskSELc2V0bachelor.cxx | 12 +++++------- PWGHF/vertexingHF/AliCFVertexingHFLctoV0bachelor.cxx | 2 +- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/PWGHF/vertexingHF/AliAnalysisTaskSELc2V0bachelor.cxx b/PWGHF/vertexingHF/AliAnalysisTaskSELc2V0bachelor.cxx index 692881f5f9c..a7af9add6cd 100644 --- a/PWGHF/vertexingHF/AliAnalysisTaskSELc2V0bachelor.cxx +++ b/PWGHF/vertexingHF/AliAnalysisTaskSELc2V0bachelor.cxx @@ -1822,11 +1822,10 @@ void AliAnalysisTaskSELc2V0bachelor::FillTheTree(AliAODRecoCascadeHF *part, AliR fCandidateVariables[80]=bachelor->Charge(); fCandidateVariables[81]=isMCparticleInFiducialAcceptance; if (fUseMCInfo) { + fCandidateVariables[82]=0; if (bachelor->GetLabel()!=-1) { AliAODMCParticle *partBachelor = dynamic_cast(mcArray->At(TMath::Abs(bachelor->GetLabel()))); - fCandidateVariables[82]=partBachelor->GetPdgCode(); - } else { - fCandidateVariables[82]=0; + if(partBachelor) fCandidateVariables[82]=partBachelor->GetPdgCode(); } } else { fCandidateVariables[82]=-1; @@ -2431,13 +2430,12 @@ Int_t AliAnalysisTaskSELc2V0bachelor::MatchToMClabelC(AliAODRecoCascadeHF *candi Int_t ndg2 = partLc->GetDaughter(1)-partLc->GetDaughter(0)+1; if (ndg2==2) return -1; - Char_t stringaCheck[100]; - sprintf(stringaCheck,">>>>>>>> %d -> ",partLc->GetPdgCode()); + TString stringaCheck = Form(">>>>>>>> %d -> ",partLc->GetPdgCode()); for(Int_t ii=0; iiAt(partLc->GetDaughter(0)+ii)); - sprintf(stringaCheck," %s %d",stringaCheck,partDau->GetPdgCode()); + stringaCheck.Append(Form(" %d",partDau->GetPdgCode())); } - printf("%s \n",stringaCheck); + printf("%s \n",stringaCheck.Data()); return indexMotherBach; diff --git a/PWGHF/vertexingHF/AliCFVertexingHFLctoV0bachelor.cxx b/PWGHF/vertexingHF/AliCFVertexingHFLctoV0bachelor.cxx index fc877ccc94d..c7c63b07a0f 100644 --- a/PWGHF/vertexingHF/AliCFVertexingHFLctoV0bachelor.cxx +++ b/PWGHF/vertexingHF/AliCFVertexingHFLctoV0bachelor.cxx @@ -914,7 +914,7 @@ Bool_t AliCFVertexingHFLctoV0bachelor::FillVectorFromMCarray(AliAODMCParticle *m AliAODMCParticle *mcPartV0DaughterNeg = dynamic_cast(fmcArray->At(fLabelArray[2])); AliAODMCParticle *mcPartDaughterV0 = 0x0; - if(!mcPartV0DaughterPos && !mcPartV0DaughterNeg) return bGenValues; + if(!mcPartV0DaughterPos || !mcPartV0DaughterNeg) return bGenValues; if (TMath::Abs(mcPartDaughterK0->GetPdgCode())==311) { Int_t daughterK0 = mcPartDaughterK0->GetDaughter(0); -- 2.43.5