workaround: Avoid geoloader memory issues

ChristianTackeGSI · dennisklein · commit e58c820d70c2 · 2024-04-16T21:07:49.000+02:00
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: #1514 See: ede7137
diff --git 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()

Original file line number	Diff line number	Diff line change
`@@ -111,6 +111,10 @@ FairRunSim::~FairRunSim()`
`111`	`111`	`// Do not point to a destructed object!`
`112`	`112`	`fginstance = nullptr;`
`113`	`113`	`}`
	`114`	`+`
	`115`	`+ /// \bug Leaks GeoLoader and related resources, prevents memory issues (probably a double free)`
	`116`	`+ /// See: https://github.com/FairRootGroup/FairRoot/issues/1514`
	`117`	`+ static_cast<void>(fGeoLoader.release());`
`114`	`118`	`}`
`115`	`119`
`116`	`120`	`FairRunSim* FairRunSim::Instance()`