[Bug 38677] New: valgrind shows invalid memory use in dlls/kernel32/tests/path.c:test_PathNameA()

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

Bug ID: 38677
           Summary: valgrind shows invalid memory use in
                    dlls/kernel32/tests/path.c:test_PathNameA()
           Product: Wine
           Version: 1.7.44
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase, valgrind
          Severity: normal
          Priority: P2
         Component: kernel32
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: Fedora

==15348== Invalid write of size 2
==15348==    at 0x7B858CAD: lstrcpynW (winbase.h:2573)
==15348==    by 0x7B859ED7: GetShortPathNameW (path.c:512)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==  Address 0x46c4922 is 0 bytes after a recently re-allocated block of
size 34 alloc'd
==15348==    at 0x7BC4A9F1: notify_alloc (heap.c:254)
==15348==    by 0x7BC4EC94: RtlAllocateHeap (heap.c:1715)
==15348==    by 0x7B84351C: HeapAlloc (heap.c:271)
==15348==    by 0x7B859C9E: GetShortPathNameW (path.c:466)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==

==15348== Invalid write of size 2
==15348==    at 0x7B858CCA: lstrcpynW (winbase.h:2575)
==15348==    by 0x7B859ED7: GetShortPathNameW (path.c:512)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==  Address 0x46c4924 is 2 bytes after a recently re-allocated block of
size 34 alloc'd
==15348==    at 0x7BC4A9F1: notify_alloc (heap.c:254)
==15348==    by 0x7BC4EC94: RtlAllocateHeap (heap.c:1715)
==15348==    by 0x7B84351C: HeapAlloc (heap.c:271)
==15348==    by 0x7B859C9E: GetShortPathNameW (path.c:466)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==

==15348== Invalid read of size 2
==15348==    at 0x7BC61692: RtlIsDosDeviceName_U (path.c:300)
==15348==    by 0x7BC628A3: RtlGetFullPathName_U (path.c:791)
==15348==    by 0x7BC61A16: RtlDosPathNameToNtPathName_U (path.c:379)
==15348==    by 0x7B83F154: FindFirstFileExW (file.c:1889)
==15348==    by 0x7B840183: FindFirstFileW (file.c:2263)
==15348==    by 0x7B859F34: GetShortPathNameW (path.c:525)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==  Address 0x46c4922 is 0 bytes after a recently re-allocated block of
size 34 alloc'd
==15348==    at 0x7BC4A9F1: notify_alloc (heap.c:254)
==15348==    by 0x7BC4EC94: RtlAllocateHeap (heap.c:1715)
==15348==    by 0x7B84351C: HeapAlloc (heap.c:271)
==15348==    by 0x7B859C9E: GetShortPathNameW (path.c:466)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==

==15348== Invalid read of size 2
==15348==    at 0x7BC60B83: strlenW (unicode.h:200)
==15348==    by 0x7BC6271E: get_full_path_helper (path.c:744)
==15348==    by 0x7BC6293E: RtlGetFullPathName_U (path.c:805)
==15348==    by 0x7BC61A16: RtlDosPathNameToNtPathName_U (path.c:379)
==15348==    by 0x7B83F154: FindFirstFileExW (file.c:1889)
==15348==    by 0x7B840183: FindFirstFileW (file.c:2263)
==15348==    by 0x7B859F34: GetShortPathNameW (path.c:525)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==  Address 0x46c4922 is 0 bytes after a recently re-allocated block of
size 34 alloc'd
==15348==    at 0x7BC4A9F1: notify_alloc (heap.c:254)
==15348==    by 0x7BC4EC94: RtlAllocateHeap (heap.c:1715)
==15348==    by 0x7B84351C: HeapAlloc (heap.c:271)
==15348==    by 0x7B859C9E: GetShortPathNameW (path.c:466)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==

==15348== Invalid read of size 2
==15348==    at 0x400AC2C: __GI_memmove (vg_replace_strmem.c:1143)
==15348==    by 0x7BC62772: get_full_path_helper (path.c:752)
==15348==    by 0x7BC6293E: RtlGetFullPathName_U (path.c:805)
==15348==    by 0x7BC61A16: RtlDosPathNameToNtPathName_U (path.c:379)
==15348==    by 0x7B83F154: FindFirstFileExW (file.c:1889)
==15348==    by 0x7B840183: FindFirstFileW (file.c:2263)
==15348==    by 0x7B859F34: GetShortPathNameW (path.c:525)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==  Address 0x46c4924 is 2 bytes after a recently re-allocated block of
size 34 alloc'd
==15348==    at 0x7BC4A9F1: notify_alloc (heap.c:254)
==15348==    by 0x7BC4EC94: RtlAllocateHeap (heap.c:1715)
==15348==    by 0x7B84351C: HeapAlloc (heap.c:271)
==15348==    by 0x7B859C9E: GetShortPathNameW (path.c:466)
==15348==    by 0x7B85A25C: GetShortPathNameA (path.c:582)
==15348==    by 0x48365BA: test_PathNameA (path.c:884)
==15348==    by 0x483D211: func_path (path.c:2225)
==15348==    by 0x4893BD4: run_test (test.h:584)
==15348==    by 0x489401C: main (test.h:666)
==15348==

etc.

[Bug 38677] valgrind shows invalid memory use in dlls/kernel32/tests/path.c:test_PathNameA()

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

Thomas Faller  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |9178d037d71e823f5d5eaa5008a
                   |                            |3cdd574377bac
             Status|NEW                         |RESOLVED
                 CC|                            |[e-mail]
         Resolution|---                         |FIXED

--- Comment #1 from Thomas Faller  ---
Fixed by 9178d037d71e823f5d5eaa5008a3cdd574377bac

[Bug 38677] valgrind shows invalid memory use in dlls/kernel32/tests/path.c:test_PathNameA()

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

Alexandre Julliard  changed:

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

--- Comment #2 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.55.