Integrated: 8341797: Fix ExceptionOccurred in jdk.jdi

Justin Lu jlu at openjdk.org
Fri Oct 11 21:35:25 UTC 2024


On Thu, 10 Oct 2024 21:56:39 GMT, Justin Lu <jlu at openjdk.org> wrote:

> Please review this PR which is part of the bigger umbrella bug: https://bugs.openjdk.org/browse/JDK-8341542.
> 
> This fixes incorrect usage of `jthrowable ExceptionOccurred(JNIEnv *env)` within _jdk.jdi_.
> This corrects instances where the return value is being treated as a boolean. Such occurrences are replaced with `jboolean ExceptionCheck(JNIEnv *env)`.
> 
> JNI Docs - ExceptionCheck
>> We introduce a convenience function to check for pending exceptions without creating a local reference to the exception object. ... Returns JNI_TRUE when there is a pending exception; otherwise, returns JNI_FALSE.

This pull request has now been integrated.

Changeset: 2db33971
Author:    Justin Lu <jlu at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2db3397187563d1821d24578247f764c372fbb4b
Stats:     11 lines in 2 files changed: 0 ins; 0 del; 11 mod

8341797: Fix ExceptionOccurred in jdk.jdi

Reviewed-by: amenkov, cjplummer

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

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


More information about the serviceability-dev mailing list