[Bug 49585] New: valgrind shows a possible leak in dlls/ntdll/env.c:read_nls_file()

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49585

Bug ID: 49585
           Summary: valgrind shows a possible leak in
                    dlls/ntdll/env.c:read_nls_file()
           Product: Wine
           Version: 5.13
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase, valgrind
          Severity: normal
          Priority: P2
         Component: ntdll
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: Gentoo

Getting a lot of spam, presumably every thread/process:
==9262== 5,512 bytes in 1 blocks are possibly lost in loss record 33 of 33
==9262==    at 0x40345DC: malloc (vg_replace_malloc.c:307)
==9262==    by 0x486401C: read_nls_file (env.c:100)
==9262==    by 0x48649EF: init_environment (env.c:961)
==9262==    by 0x48733BA: __wine_main (loader.c:1710)
==9262==    by 0x7D0017D7: main (main.c:285)
==9262==

[Bug 49585] valgrind shows a possible leak in dlls/ntdll/env.c:read_nls_file()

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49585

--- Comment #1 from Alexandre Julliard  ---
That's not a leak, we need that data for the process lifetime.

[Bug 49585] valgrind shows a possible leak in dlls/ntdll/env.c:read_nls_file()

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49585

Zebediah Figura  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from Zebediah Figura  ---
I guess it's probably not worthwhile, but maybe we can clean up such things at
process exit...?

[Bug 49585] valgrind shows a possible leak in dlls/ntdll/env.c:read_nls_file()

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49585

--- Comment #3 from Alexandre Julliard  ---
We could but I don't think it would be an improvement.

[Bug 49585] valgrind shows a possible leak in dlls/ntdll/env.c:read_nls_file()

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49585

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |WONTFIX
             Status|NEW                         |RESOLVED

--- Comment #4 from Austin English  ---
(In reply to Alexandre Julliard from comment #3)
> We could but I don't think it would be an improvement.

Thanks, marked as ignored:
https://github.com/austin987/wine-valgrind-scripts/commit/e1ddc4588110a998c7...

[Bug 49585] valgrind shows a possible leak in dlls/ntdll/env.c:read_nls_file()

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=49585

Gijs Vermeulen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #5 from Gijs Vermeulen  ---
Closing wontfix.