From ecc4d31aa5869bc89b57034dced4b3136071cb60 Mon Sep 17 00:00:00 2001 From: Christoph Oelckers Date: Thu, 4 Mar 2021 16:55:50 +0100 Subject: [PATCH] - fixed memory leak in UDMF loader. When the extsector allocation was moved, this hadn't been adjusted yet, it still allocated its own buffer which never got freed again. --- src/maploader/udmf.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/maploader/udmf.cpp b/src/maploader/udmf.cpp index 0b5e5f6424..b96f2f71b6 100644 --- a/src/maploader/udmf.cpp +++ b/src/maploader/udmf.cpp @@ -2318,10 +2318,10 @@ public: // Create the real sectors Level->sectors.Alloc(ParsedSectors.Size()); memcpy(&Level->sectors[0], &ParsedSectors[0], Level->sectors.Size() * sizeof(sector_t)); - Level->sectors[0].e = new extsector_t[Level->sectors.Size()]; + Level->extsectors.Alloc(Level->sectors.Size()); for(unsigned i = 0; i < Level->sectors.Size(); i++) { - Level->sectors[i].e = &Level->sectors[0].e[i]; + Level->sectors[i].e = &Level->extsectors[i]; } // Now create the scrollers. for (auto &scroll : UDMFScrollers)