RFR: JDK-8318955: Add ReleaseIntArrayElements in Java_sun_awt_X11_XlibWrapper_SetBitmapShape XlbWrapper.c to early return [v2]
Phil Race
prr at openjdk.org
Mon Oct 30 20:48:31 UTC 2023
On Mon, 30 Oct 2023 09:15:00 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> There is an early return in Java_sun_awt_X11_XlibWrapper_SetBitmapShape XlbWrapper.c that seems to miss a ReleaseIntArrayElements call.
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>
> Add JNU_ThrowOutOfMemoryError
src/java.desktop/unix/native/libawt_xawt/xawt/XlibWrapper.c line 2303:
> 2301: if (!pRect) {
> 2302: (*env)->ReleaseIntArrayElements(env, bitmap, values, JNI_ABORT);
> 2303: JNU_ThrowOutOfMemoryError(env, "malloc failed");
I think the throw is unnecessary. You will get some incorrect rendering, but that's all.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16392#discussion_r1376781142
More information about the client-libs-dev
mailing list