RFR: 8362889: [GCC static analyzer] leak in libstringPlatformChars.c [v3]
Roger Riggs
rriggs at openjdk.org
Tue Jul 22 13:37:26 UTC 2025
On Tue, 22 Jul 2025 08:32:06 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> This is reported by the gcc static analyzer (-fanalyzer); it is just test coding so not very critical but probably should still be adjusted .
>>
>>
>> /jdk/test/jdk/java/lang/String/nativeEncoding/libstringPlatformChars.c:74:12: warning: leak of 'str' [CWE-401] [-Wanalyzer-malloc-leak]
>> 74 | return JNU_NewStringPlatform(env, str);
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>
> Adjust Java_StringPlatformChars_newString
test/jdk/java/lang/String/nativeEncoding/libstringPlatformChars.c line 67:
> 65: str = (char*)malloc(len + 1);
> 66: if (str == NULL) {
> 67: return NULL;
Six of one, half-dozen of the other.
You'll need to ReleasePrimitiveArrayCritical here or revert to the previous version.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26415#discussion_r2222560525
More information about the core-libs-dev
mailing list