Skip to content

Fix memory leak in IncidentSvc and more

Rosen Matev requested to merge rmatev/Gaudi:rm-sanitizers into master
  • !1164 (merged) introduced a regular clean up of incidents but the memory was not freed
  • Fix tiny memory leak in HiveTestAlgorithm
  • Allow for multiple sanitizers to be used at once by setting PRELOAD_SANITIZER_LIB (e.g. to libasan.so:libubsan.so).
  • Also, sanitizers are ignored when LD_PRELOAD is not empty and --profilerName=jemalloc has no effect when using a sanitizer.

There is some UB that I don't know how to fix and don't manage to suppress, see #201 (closed)

Together with the update of suppressed leaks in lhcb-core/lcg-toolchains!64 (merged), this should improve LHCb's sanitizer builds. I suppose it would be nice to also run sanitizers in the gitlab CI here.

/cc @fwinkl

Edited by Rosen Matev

Merge request reports