John,

1) 332-333 can be replaced w/ a deleteGSSOID(nameType) call. Also, with this deleteGSSOID(nameType) call, we should also add the following line:
    resetGSSBuffer(env, jnameVal, &nameVal);

2) I think the ExceptionCheck block on line 932 should also be enhanced w/ resetGSSBuffer(env, jinToken, &inToken) call as well as free(cb) call. Same goes for another ExceptionCheck blocks on line 984, 998, 1024.

Thanks,
Valerie
On 05/14/13 19:19, John Zavgren wrote:
Greetings: Please review the following change to the file: jdk/src/share/native/sun/security/jgss/wrapper/GSSLibStub.c that fixes two memory leaks. http://cr.openjdk.java.net/~jzavgren/8014307/webrev.01/ Thanks! John Zavgren

Reply via email to