RFR: 7124710: interleaved RedefineClasses() and RetransformClasses() calls may have a problem [v3]

Alex Menkov amenkov at openjdk.org
Wed Sep 7 20:09:05 UTC 2022


> The problem is RedefineClasses does not update cached_class_bytes, so subsequent RetransformClasses gets obsolete class bytes (this are testcases 3-6 from the new test)
> 
> cached_class_bytes are set when an agent instruments the class from ClassFileLoadHook.
> After successful RedefineClasses it should be reset.
> The fix updates ClassFileLoadHook caller to not use old cached_class_bytes with RedefineClasses (if some agent instruments the class, new cached_class_bytes are allocated for scratch_class) and updates cached_class_bytes after successful RedefineClasses or RetransformClasses.

Alex Menkov has updated the pull request incrementally with one additional commit since the last revision:

  updated test. comments, code reorg

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/10032/files
  - new: https://git.openjdk.org/jdk/pull/10032/files/d86ae7fa..9bc08346

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10032&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10032&range=01-02

  Stats: 303 lines in 2 files changed: 129 ins; 104 del; 70 mod
  Patch: https://git.openjdk.org/jdk/pull/10032.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10032/head:pull/10032

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


More information about the serviceability-dev mailing list