Skip to content

Commit

Permalink
workaround: Avoid geoloader memory issues
Browse files Browse the repository at this point in the history
Not leaking fGeoLoader on FairRunSim seems to cause
some memory issues (probbaly a double free/delete).
Until we find it, let's leak the Geoloader, as before.

See: FairRootGroup#1514
See: ede7137
  • Loading branch information
ChristianTackeGSI committed Apr 5, 2024
1 parent 66709b8 commit 3b3d868
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions fairroot/base/steer/FairRunSim.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,10 @@ FairRunSim::~FairRunSim()
// Do not point to a destructed object!
fginstance = nullptr;
}

/// \bug Leaks GeoLoader and related resources, prevents memory issues (probably a double free)
/// See: https://github.com/FairRootGroup/FairRoot/issues/1514
static_cast<void>(fGeoLoader.release());
}

FairRunSim* FairRunSim::Instance()
Expand Down

0 comments on commit 3b3d868

Please sign in to comment.