RFR: 8284855: Update needed to Cleaners added to jdk.crypto.cryptoki [v5]

Xue-Lei Andrew Fan xuelei at openjdk.java.net
Fri Apr 15 15:50:19 UTC 2022


> This is an effort to fix a problem introduced in the fix for [JDK-8284368](https://bugs.openjdk.java.net/browse/JDK-8284368), which replaced the finalizers in jdk.crypto.cryptoki with Cleaners.  However, there is a problem with the code changes. The Runnables registered with Cleaner refer to the object being registered ('this'). Meaning, the Cleaner mechanism will keep the objects reachable, preventing them from being cleaned and collected.

Xue-Lei Andrew Fan has updated the pull request incrementally with one additional commit since the last revision:

  add a reference to the clean up method

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8248/files
  - new: https://git.openjdk.java.net/jdk/pull/8248/files/ec1641ae..650bcb9b

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8248&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8248&range=03-04

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8248.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8248/head:pull/8248

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



More information about the security-dev mailing list