On Sun, 1 May 2022 19:31:01 GMT, Phil Race <p...@openjdk.org> wrote:

> A tool checking for JNI errors complains that the call to 
> JNU_NewStringPlatform(..)
> might throw an exception and subsequent JNI code isn't making sure of that.
> Clear the exception so the error handling code can do its thing.

src/java.desktop/unix/native/common/awt/CUPSfuncs.c line 249:

> 247:             utf_str = JNU_NewStringPlatform(env, dests[i].name);
> 248:             if (utf_str == NULL) {
> 249:                 (*env)->ExceptionClear(env);

Shouldn't we also need to do same in 
https://github.com/openjdk/jdk/blob/master/src/java.desktop/unix/native/common/awt/CUPSfuncs.c#L357,
 L367, L381 since we do at L348

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

PR: https://git.openjdk.java.net/jdk/pull/8491

Reply via email to