From 3b3d8682e0416cb7d0a07c8deb0e9ff711eb95da Mon Sep 17 00:00:00 2001
From: Christian Tacke <58549698+ChristianTackeGSI@users.noreply.github.com>
Date: Fri, 5 Apr 2024 18:29:34 +0200
Subject: [PATCH] workaround: Avoid geoloader memory issues

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: https://github.com/FairRootGroup/FairRoot/issues/1514
See: ede7137bd698df6ba9a6b4e6807e8225e68e121b
---
 fairroot/base/steer/FairRunSim.cxx | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/fairroot/base/steer/FairRunSim.cxx b/fairroot/base/steer/FairRunSim.cxx
index bc86543c8b..c3b9d5cc93 100644
--- a/fairroot/base/steer/FairRunSim.cxx
+++ b/fairroot/base/steer/FairRunSim.cxx
@@ -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()