Fix memory leak in IncidentSvc and more

• !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

added lhcb-sanitizers label

added 1 commit

• 366ea174 - Allow for multiple sanitizers to be used at once

Compare with previous version

changed the description

• [2021-07-30 00:55] Validation started with lhcb-sanitizers#917
• [2021-07-31 00:08] Validation started with lhcb-gaudi-head#2999
• [2021-07-31 00:19] Validation started with lhcb-sanitizers#918
• [2021-07-31 00:20] Validation started with lhcb-run2-gaudi-head#177
• [2021-08-03 00:04] Validation started with lhcb-gaudi-head#3000
• [2021-08-03 00:30] Validation started with lhcb-run2-gaudi-head#178
• [2021-08-03 00:32] Validation started with lhcb-sanitizers#919
• [2021-08-03 00:37] Validation started with lhcb-run2-gaudi-head#178
• [2021-08-04 00:07] Validation started with lhcb-gaudi-head#3001
• [2021-08-04 00:58] Validation started with lhcb-run2-gaudi-head#179
• [2021-08-04 01:00] Validation started with lhcb-sanitizers#920
• [2021-08-05 00:04] Validation started with lhcb-gaudi-head#3002
• [2021-08-05 00:46] Validation started with lhcb-sanitizers#921
• [2021-08-06 00:09] Validation started with lhcb-gaudi-head#3003
• [2021-08-06 00:40] Validation started with lhcb-sanitizers#922
• [2021-08-07 00:07] Validation started with lhcb-gaudi-head#3004
• [2021-08-07 00:55] Validation started with lhcb-sanitizers#923
• [2021-08-07 01:03] Validation started with lhcb-gaudi-head#3004
• [2021-08-07 01:23] Validation started with lhcb-gaudi-head#3004
• [2021-08-07 01:53] Validation started with lhcb-sanitizers#923
• [2021-08-08 00:03] Validation started with lhcb-gaudi-head#3005
• [2021-08-08 00:27] Validation started with lhcb-sanitizers#924
• [2021-08-08 00:41] Validation started with lhcb-sanitizers#924
• [2021-08-10 00:04] Validation started with lhcb-gaudi-head#3006
• [2021-08-10 00:36] Validation started with lhcb-sanitizers#925
• [2021-08-11 00:06] Validation started with lhcb-gaudi-head#3007
• [2021-08-11 00:50] Validation started with lhcb-run2-gaudi-head#180
• [2021-08-11 00:54] Validation started with lhcb-sanitizers#926
• [2021-08-12 00:04] Validation started with lhcb-gaudi-head#3008
• [2021-08-12 00:35] Validation started with lhcb-sanitizers#927
• [2021-08-13 00:04] Validation started with lhcb-gaudi-head#3009
• [2021-08-13 00:39] Validation started with lhcb-sanitizers#928
• [2021-08-14 00:04] Validation started with lhcb-gaudi-head#3010
• [2021-08-14 00:36] Validation started with lhcb-sanitizers#929
• [2021-08-17 00:04] Validation started with lhcb-gaudi-head#3011
• [2021-08-17 00:31] Validation started with lhcb-sanitizers#930
• [2021-08-18 00:06] Validation started with lhcb-gaudi-head#3012
• [2021-08-18 01:04] Validation started with lhcb-run2-gaudi-head#181
• [2021-08-18 01:14] Validation started with lhcb-sanitizers#931
• [2021-08-19 00:04] Validation started with lhcb-gaudi-head#3013
• [2021-08-19 00:57] Validation started with lhcb-sanitizers#932
• [2021-08-19 01:04] Validation started with lhcb-run2-gaudi-head#182
• [2021-08-20 00:07] Validation started with lhcb-gaudi-head#3014
• [2021-08-20 01:07] Validation started with lhcb-run2-gaudi-head#183
• [2021-08-20 01:26] Validation started with lhcb-sanitizers#933
• [2021-08-21 00:08] Validation started with lhcb-gaudi-head#3015
• [2021-08-21 00:53] Validation started with lhcb-run2-gaudi-head#184
• [2021-08-21 01:02] Validation started with lhcb-sanitizers#934
• [2021-08-22 00:03] Validation started with lhcb-gaudi-head#3016
• [2021-08-22 00:28] Validation started with lhcb-sanitizers#935
• [2021-08-24 00:05] Validation started with lhcb-gaudi-head#3017
• [2021-08-24 00:51] Validation started with lhcb-run2-gaudi-head#185
• [2021-08-24 00:52] Validation started with lhcb-sanitizers#936
• [2021-08-25 00:03] Validation started with lhcb-gaudi-head#3018
• [2021-08-25 00:52] Validation started with lhcb-run2-gaudi-head#186
• [2021-08-25 00:59] Validation started with lhcb-sanitizers#937
• [2021-08-26 00:03] Validation started with lhcb-gaudi-head#3019
• [2021-08-26 00:41] Validation started with lhcb-sanitizers#938
• [2021-08-27 00:05] Validation started with lhcb-gaudi-head#3020
• [2021-08-27 00:40] Validation started with lhcb-sanitizers#939
• [2021-08-28 00:05] Validation started with lhcb-gaudi-head#3021
• [2021-08-28 00:39] Validation started with lhcb-sanitizers#940
• [2021-08-30 00:04] Validation started with lhcb-gaudi-head#3022
• [2021-08-30 00:33] Validation started with lhcb-sanitizers#941
• [2021-09-01 00:05] Validation started with lhcb-gaudi-head#3023
• [2021-09-01 00:49] Validation started with lhcb-run2-gaudi-head#187
• [2021-09-01 00:59] Validation started with lhcb-sanitizers#942
• [2021-09-02 00:07] Validation started with lhcb-gaudi-head#3024
• [2021-09-02 00:49] Validation started with lhcb-sanitizers#943

added 1 commit

• 00a1865d - Allow for multiple sanitizers to be used at once

Compare with previous version

mentioned in commit lhcb-core/lcg-toolchains@2dc01dca

mentioned in merge request lhcb-core/lcg-toolchains!65 (merged)

this had the desired effect in the nightly build

assigned to @clemenci

changed milestone to %v36r1

added lhcb-gaudi-head label

I just added the label lhcb-gaudi-head so that @fwinkl can check if there's an impact in ATLAS.

mentioned in commit lhcb-core/lcg-toolchains@0d068756

added 1 commit

• c4ba0b57 - Allow for multiple sanitizers to be used at once

Compare with previous version

added 1 commit

• ab211de2 - Allow for multiple sanitizers to be used at once

Compare with previous version

added 1 commit

• 505bce45 - Allow for multiple sanitizers to be used at once

Compare with previous version

resolved all threads