Hi Fred, I ran the repro on a build with your fix applied and it appears that the leak is fixed. Thanks for finding it so quick :)
/Mikael On Friday 10 February 2012 10.27.30 Frederic Parain wrote: > Here's a small fix (one line) for CR 7143760 Memory leak in > GarbageCollectionNotifications > > There's a missing HandleMark at the beginning of the > GCNotifier::sendNotificatin() method. Without this HandleMark, all > handles used when creating GC notifications are kept alive causing a > double leak: in the Java heap and in the thread local handle area of the > service thread. > > Here's the CR: > http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7143760 > (Warning, the changeset referenced in the CR is not the > one containing the original bug). > > Here's the webrev: > http://cr.openjdk.java.net/~fparain/7143760/webrev.00/ > > Thanks, > > Fred
