On Wed, 3 Apr 2024 18:18:37 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:

>> We do have checks for jGetFunctionList != NULL before calling 
>> ReleaseStringUTFChars() with it. So, this shouldn't be an issue?
>
> this will be an issue if `jGetFunctionList != NULL` and `dlopen` fails, 
> resulting in `goto cleanup` before `getFunctionListStr` is reassigned.

Hmm, ok, I will change it. Thanks~

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18588#discussion_r1550333866

Reply via email to