RFR: 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() [v2]

Alexey Ivanov aivanov at openjdk.java.net
Tue Mar 8 21:18:07 UTC 2022


On Tue, 8 Mar 2022 21:12:33 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

>>> I think `NewStringUTF()` can throw OOM and also return `NULL`, just which one you pick.
>> 
>> Yes.
>> 
>> But we're discussing whether a check for exception is necessary after `(*env)->SetObjectArrayElement`.
>
> `SetObjectArrayElement()` may throw `ArrayIndexOutOfBoundsException` and `ArrayStoreException`, but I don't see it is possible here.

That's exactly my point. Therefore the check for exception after `SetObjectArrayElement` is unnecessary.

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

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


More information about the core-libs-dev mailing list