On Fri, 11 Jul 2025 13:04:37 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:

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

Hi I noticed that one too !  Seems we have to adjust it too , I first thought 
that it is not needed there but it is. 
Not sure why the GCC analyzer reports only the first .
Unfortunately  one of the calls to ParseLocale does not even check the return 
value , so it is all a bit 'shaky' .

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

PR Comment: https://git.openjdk.org/jdk/pull/26264#issuecomment-3065896882

Reply via email to