RFR: 8359268: 3 JNI exception pending defect groups in 2 files [v2]
Daniel Jeliński
djelinski at openjdk.org
Tue Jun 24 20:04:30 UTC 2025
On Tue, 24 Jun 2025 16:14:13 GMT, Michael McMahon <michaelm at openjdk.org> wrote:
>> Hi,
>>
>> This change is a follow on from https://bugs.openjdk.org/browse/JDK-8348986
>> In a couple of places, the new native code in that fix can potentially throw
>> an exception while another exception is pending. This needs to be fixed.
>>
>> Thanks,
>> Michael
>
> Michael McMahon has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>
> - Merge branch 'JNI-8359268' of /Users/mimcmah/git/jdk/open into JNI-8359268
> - Merge branch 'master' into JNI-8359268
> - net_util.c update
> - Merge branch 'master' into JNI-8359268
> - windows compile error
> - update
> - impl fix
src/java.base/windows/native/libnet/Inet4AddressImpl.c line 93:
> 91: int enh = getEnhancedExceptionsAllowed(env);
> 92: if (enh == ENH_INIT_ERROR && (*env)->ExceptionCheck(env)) {
> 93: return NULL;
should it be `goto cleanupAndReturn;` instead?
src/java.base/windows/native/libnet/Inet6AddressImpl.c line 88:
> 86: int enh = getEnhancedExceptionsAllowed(env);
> 87: if (enh == ENH_INIT_ERROR && (*env)->ExceptionCheck(env)) {
> 88: return NULL;
should it be `goto cleanupAndReturn;` instead?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25898#discussion_r2164789558
PR Review Comment: https://git.openjdk.org/jdk/pull/25898#discussion_r2164793497
More information about the net-dev
mailing list