Don't leak weak pointers

The free() call after the return is never executed, so we have to move it
inside the locked() block of code.

diff --git a/rt/gc/cdgc/gc.d b/rt/gc/cdgc/gc.d
index e6b3c11c8b330963329571de190f844711adf466..98e5899fcf3e41ad1cd5c4b4abf37642d9cdee9c 100644 (file)
--- a/rt/gc/cdgc/gc.d
+++ b/rt/gc/cdgc/gc.d
@@ -1981,8 +1981,8 @@ void weakpointerDestroy( void* p )
         return locked!(void, () {
             if (wp.reference)
                 rt_detachDisposeEvent(wp.reference, &wp.ondestroy);
+            cstdlib.free(wp);
         })();
-        cstdlib.free(wp);
     }
 }