RFR: 8284490: Remove finalizer method in java.security.jgss [v5]

Xue-Lei Andrew Fan xuelei at openjdk.java.net
Wed Apr 20 15:54:08 UTC 2022


On Wed, 20 Apr 2022 15:20:15 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

> Also should there be some synchronized block?

I thought about the synchronization issue.  It looks like this class is not multiple-threads safe, and thus the upper layer will take care of the synchronization.  So I did not add synchronization in this class update.

> should pContext be set to 0 before calling cleanable.clean() to make sure cleanable.clean() is not called twice? 

Cleanable.clean() will work one time only, no matter how many time it is called.  But it looks like better to make a change here.

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

PR: https://git.openjdk.java.net/jdk/pull/8136



More information about the security-dev mailing list