Skip to content
Snippets Groups Projects
Commit 00de0507 authored by Bertrand Martin Dit Latour's avatar Bertrand Martin Dit Latour Committed by Jiri Masik
Browse files

TrigT2CaloCommon: fix memory leak in TrigCaloDataAccessSvc

TrigT2CaloCommon: fix memory leak in TrigCaloDataAccessSvc

Hello,

This MR fixes a memory leak in TrigCaloDataAccessSvc found by valgrind (ATR-30983).
In `lateInit`, there are about 250 cells created with `new`, but memory is not released in `finalize` prior to the deletion of `cache->fullcont`.
In this MR, a vector is added to keep track of the position of these cells in the container, such that we can free memory in `finalize`.
Valgrind confirms this fixes the memory leak. Note that there is no leak in the event loop.

Cheers,
Bertrand
parent 2a76f8bd
No related branches found
No related tags found
2 merge requests!787692025-03-24: merge of 24.0 into main,!78737TrigT2CaloCommon: fix memory leak in TrigCaloDataAccessSvc
Loading
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