Skip to content

loKi::Hybrid::DTFDict - Default initialise m_usePV flag

Christopher Rob Jones requested to merge LoKiArrayFunctors-DTFDict-InitUsePVFlag into master

Addresses sanitizer error

(build/Phys/LoKiArrayFunctors)$  /cvmfs/lhcb.cern.ch/lib/var/lib/LbEnv/471/stable/x86_64-centos7/bin/xenv --xml build/config/Phys-build.xenv Gaudi/InstallArea/x86_64-centos7-gcc8-dbg+ubsan/bin/genconf.exe -o build/Phys/LoKiArrayFunctors/genConf/LoKiArrayFunctors -p LoKiArrayFunctors --configurable-module=GaudiKernel.Proxy --configurable-default-name=Configurable.DefaultName --configurable-algorithm=ConfigurableAlgorithm --configurable-algtool=ConfigurableAlgTool --configurable-auditor=ConfigurableAuditor --configurable-service=ConfigurableService -i LoKiArrayFunctors 
Gaudi/GaudiKernel/GaudiKernel/ToStream.h:338:15: runtime error: load of value 160, which is not a valid value for type 'bool'
    #0 0x7f914e299019 in std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > Gaudi::Utils::toString<bool>(bool const&) Gaudi/GaudiKernel/GaudiKernel/ToStream.h:338
    #1 0x7f9143da1cd4 in Gaudi::Details::Property::DefaultStringConverterImpl<bool>::toString[abi:cxx11](bool const&) Gaudi/InstallArea/x86_64-centos7-gcc8-dbg+ubsan/include/GaudiKernel/Property.h:171
    #2 0x7f9143da1cd4 in Gaudi::Property<bool&, Gaudi::Details::Property::NullVerifier, Gaudi::Details::Property::UpdateHandler>::toString[abi:cxx11]() const Gaudi/InstallArea/x86_64-centos7-gcc8-dbg+ubsan/include/GaudiKernel/Property.h:684
    #3 0x444e6c in configGenerator::pythonizeValue(Gaudi::Details::PropertyBase const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&) Gaudi/GaudiKernel/src/Util/genconf.cpp:721
    #4 0x448544 in genComponent Gaudi/GaudiKernel/src/Util/genconf.cpp:681
    #5 0x450bfe in configGenerator::genConfig(std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) Gaudi/GaudiKernel/src/Util/genconf.cpp:568

@erodrigu @rmatev Do you not in Phys enforce clang code formatting rules ? I automatically clang formatted the file here, which has lead to a number of additional diffs. If not I think you should, to mirror what is done in LHCb, Rec etc.

Merge request reports