RFR: 8361959: [GCC static analyzer] java_props_md.c leak of 'temp' variable is reported [v2]

Aleksey Shipilev shade at openjdk.org
Tue Jul 15 07:03:45 UTC 2025


On Sat, 12 Jul 2025 17:36:54 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> The following is reported when building with the gcc static analyzer (-fanalyzer) :
>> 
>> 
>> /jdk/src/java.base/unix/native/libjava/java_props_md.c:244:17: warning: leak of 'temp' [CWE-401] [-Wanalyzer-malloc-leak]
>>   244 | return 0;
>> 
>> 
>> Seems we have to free temp in an early return.
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   free before another early return

There is another instance a few lines below. See around where `free(encoding_variant);` happen.

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

Marked as reviewed by shade (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26264#pullrequestreview-3018962127


More information about the core-libs-dev mailing list