Request for Review (XS): 7143760 Memory leak in GarbageCollectionNotifications

[Mikael Gerdin]

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