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