]> git.uio.no Git - u/mrichter/AliRoot.git/commit
Jan, 15th, 2013 : Coverity fix - defect 11329
authoramaire <amaire@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 15 Jan 2013 18:03:24 +0000 (18:03 +0000)
committeramaire <amaire@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 15 Jan 2013 18:03:24 +0000 (18:03 +0000)
commit7475515fcd987039f7cda612eca2ab922ca49de4
treee7bb5a34c6f88d928858c1984ccf6255839d9ee2
parent0ac98da8cde3893d869d78b08784abbf87486fd9
Jan, 15th, 2013 : Coverity fix - defect 11329

Fix in the number of priors initialised for AliPID
https://alicoverity.cern.ch:8443/sourcebrowser.htm?projectId=10001#mergedDefectId=11329

Double_t lPriorsGuessXi   [AliPID::kSPECIESCN] = {0, 0, 2, 0, 1,  0,0,0,0};
Double_t lPriorsGuessOmega[AliPID::kSPECIESCN] = {0, 0, 1, 1, 1,  0,0,0,0};
            // If the boolean for charged option is activated in AliPID::SetPriors, only the first 9 double_t in the array are read anyway.
            // But Coverity sniffs an issue (a priori, previous implementation was safe but the logic is a bit twisted; Coverity is not totally wrong).
            // Use AliPID::kSPECIESCN (=14) is size of the array + initialise what I am interested in (charged particle = 9 first particles)
            // The rest of the table will be initialised to 0 anyway.

// Combined VO-positive-daughter PID
AliPID pPidXi;         pPidXi.SetPriors(    lPriorsGuessXi    , kTRUE); // kTRUE = for charged particle PID
AliPID pPidOmega;      pPidOmega.SetPriors( lPriorsGuessOmega , kTRUE); // kTRUE = for charged particle PID
PWGLF/STRANGENESS/Cascades/AliAnalysisTaskCheckCascade.cxx
PWGLF/STRANGENESS/Cascades/AliAnalysisTaskCheckPerformanceCascade.cxx