Re: RFR JDK-8008107

2013-02-19 Thread Sean Mullan
Looks good to me too. --Sean On 02/19/2013 12:42 PM, Chris Hegarty wrote: Looks ok to me. -Chris. On 02/19/2013 05:16 PM, John Zavgren wrote: Greetings: I posted a webrev image: http://cr.openjdk.java.net/~jzavgren/8008107/webrev.01/, of a change that I made to the native source code file:

Re: RFR JDK-8008107

2013-02-19 Thread Chris Hegarty
Looks ok to me. -Chris. On 02/19/2013 05:16 PM, John Zavgren wrote: Greetings: I posted a webrev image: http://cr.openjdk.java.net/~jzavgren/8008107/webrev.01/, of a change that I made to the native source code file: jdk/src/share/native/sun/security/pkcs11/wrapper/p11_convert.c There is a b

RFR JDK-8008107

2013-02-19 Thread John Zavgren
Greetings: I posted a webrev image: http://cr.openjdk.java.net/~jzavgren/8008107/webrev.01/, of a change that I made to the native source code file: jdk/src/share/native/sun/security/pkcs11/wrapper/p11_convert.c There is a block of code in this file, around line 685, that attempts to free memor