> Fixed three test files that fail with -Xcheck:jni on debug builds:
> 
> popframe007.cpp: CallStaticVoidMethod in native B() was called with the 
> declaring class (popframe007) instead of TestThread where the method ID 
> belongs. Stored TestThread class as a global ref in getReady() and used it in 
> B()
> 
> aod.cpp: nsk_aod_agentLoaded() and nsk_aod_agentFinished() call 
> CallStaticVoidMethod without checking for pending exceptions. Added 
> nsk_aod_checkJNIException wrapper that describes, clears, and fails on 
> unexpected exceptions. Fixes all 13 AttachOnDemand tests
> 
> retransform003.cpp: ClassFileLoadHook callback calls CallObjectMethod and 
> CallStaticObjectMethod without checking for pending exceptions. Added 
> ExceptionCheck/ExceptionClear after both calls
> 
> bi02t001 and bi03t001 pass without changes, NMT corruption issue from JDK 22 
> appears resolved on JDK 27
> 
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Shiv Shah has updated the pull request incrementally with one additional commit 
since the last revision:

  Remove unnecessary return check per review

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/30711/files
  - new: https://git.openjdk.org/jdk/pull/30711/files/dfda475c..f5b31602

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

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

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

Reply via email to