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