RFR: 8303809: Dispose context in SPNEGO NegotiatorImpl

Daniel Jeliński djelinski at openjdk.org
Wed Mar 8 17:19:23 UTC 2023


On Wed, 8 Mar 2023 09:05:19 GMT, Alexey Bakhtin <abakhtin at openjdk.org> wrote:

> This patch fixes a possible native memory leak in case of a custom native GSS provider.
> The actual leak was reported in production.
> 
> sun/security/jgss, sun/security/krb5, sun/net/www/protocol/http jtreg tests are passed

ah, you're right, `cleanable` has its own copy of `pContext`. You can disregard my previous comment.

But then, is there any other place where we could be leaking native memory? As far as I could tell, this patch only releases the memory earlier, it doesn't fix any leaks. Am I missing something?

-------------

PR: https://git.openjdk.org/jdk/pull/12920



More information about the security-dev mailing list