Re: [9] RFR 8025669: [parfait] warning from b62 for jdk.src.solaris.native.com.sun.security.auth.module: memory leak

2014-02-05 Thread Chris Hegarty
Looks good to me Vincent. -Chris. On 5 Feb 2014, at 17:06, Vincent Ryan wrote: > Please review this fix for a potential memory leak in JAAS native code on > Solaris: > > Webrev: http://cr.openjdk.java.net/~vinnie/8025669/webrev.00/ > > The fix ensures that the calloc’d memory always gets fre

Re: [9] RFR 8025669: [parfait] warning from b62 for jdk.src.solaris.native.com.sun.security.auth.module: memory leak

2014-02-05 Thread Alan Bateman
On 05/02/2014 17:06, Vincent Ryan wrote: Please review this fix for a potential memory leak in JAAS native code on Solaris: Webrev: http://cr.openjdk.java.net/~vinnie/8025669/webrev.00/ The fix ensures that the calloc’d memory always gets freed. Thanks. This looks okay to me. -Alan.

[9] RFR 8025669: [parfait] warning from b62 for jdk.src.solaris.native.com.sun.security.auth.module: memory leak

2014-02-05 Thread Vincent Ryan
Please review this fix for a potential memory leak in JAAS native code on Solaris: Webrev: http://cr.openjdk.java.net/~vinnie/8025669/webrev.00/ The fix ensures that the calloc’d memory always gets freed. Thanks.