Skip to content
Snippets Groups Projects
Commit dfccc204 authored by John Chapman's avatar John Chapman
Browse files

Add Bkg_TruthEvent to the list of known McEventCollection names in HepMcParticleLink.cxx

The pre-mixing jobs rename the output `McEventCollection` to `Bkg_TruthEvent` rather than `TruthEvent`.
`HepMcParticleLink` doesn't know that this is a possible collection name that it should check for so
all links are broken and the attempts to find the `McEventCollection` in the T->P converter methods
always return `nullptr`. This causes the above crash as there is currently no protection against
dereferencing a null `McEventCollection` pointer. Adding `Bkg_TruthEvent` to the list of known
`McEventCollection` names in `HepMcParticleLink.cxx` should fix ATLASSIM-4539.
parent 90fd25cf
No related branches found
No related tags found
No related merge requests found
/*
Copyright (C) 2002-2019 CERN for the benefit of the ATLAS collaboration
Copyright (C) 2002-2020 CERN for the benefit of the ATLAS collaboration
*/
......@@ -26,14 +26,14 @@ namespace {
/**
* @brief StoreGate keys to try for each EBC_EVCOLL enum.
*/
constexpr static int NKEYS = 4;
constexpr static int NKEYS = 5;
const
std::string s_keys[EBC_NCOLLKINDS][NKEYS] =
{
{"TruthEvent","G4Truth","GEN_AOD","GEN_EVENT"},
{"TruthEvent_PU","G4Truth_PU","GEN_AOD_PU","GEN_EVENT_PU"},
{"TruthEvent_HighPtPU","G4Truth_HighPtPU","GEN_AOD_HighPtPU","GEN_EVENT_HighPtPU"},
{"TruthEvent_Cavern","G4Truth_Cavern","GEN_AOD_Cavern","GEN_EVENT_Cavern"},
{"TruthEvent","G4Truth","GEN_AOD","GEN_EVENT","Bkg_TruthEvent"},
{"TruthEvent_PU","G4Truth_PU","GEN_AOD_PU","GEN_EVENT_PU","Bkg_TruthEvent_PU"},
{"TruthEvent_HighPtPU","G4Truth_HighPtPU","GEN_AOD_HighPtPU","GEN_EVENT_HighPtPU","Bkg_TruthEvent_HighPtPU"},
{"TruthEvent_Cavern","G4Truth_Cavern","GEN_AOD_Cavern","GEN_EVENT_Cavern","Bkg_TruthEvent_Cavern"},
};
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment