]> OCCT Git - occt-copy.git/commitdiff
0031844: Coding - memory leak in Standard_MMgrFactory CR31844
authorkgv <kgv@opencascade.com>
Tue, 13 Oct 2020 04:50:19 +0000 (07:50 +0300)
committerkgv <kgv@opencascade.com>
Tue, 13 Oct 2020 04:50:19 +0000 (07:50 +0300)
Added missing deletion (previously commented without a hint why).

src/Standard/Standard.cxx

index 78b25e1cb771e8916585cfed94288123a5314753..bed42994f2945f5974d7a3f81132074f4a2cc0b8 100644 (file)
@@ -179,8 +179,11 @@ Standard_MMgrFactory::Standard_MMgrFactory()
 
 Standard_MMgrFactory::~Standard_MMgrFactory()
 {
-  if (  myFMMgr )
+  if (myFMMgr != NULL)
+  {
     myFMMgr->Purge(Standard_True);
+    delete myFMMgr;
+  }
 }
 
 //=======================================================================
@@ -208,7 +211,7 @@ Standard_MMgrFactory::~Standard_MMgrFactory()
 // WNT MSVC++) to put destructing function in code segment that is called
 // after destructors of other (even static) objects. However, this is not 
 // done by the moment since it is compiler-dependent and there is no guarantee 
-// thatsome other object calling memory manager is not placed also in that segment...
+// that some other object calling memory manager is not placed also in that segment...
 //
 // Note that C runtime function atexit() could not help in this problem 
 // since its behaviour is the same as for destructors of static objects