[Bug 36187] New: valgrind shows lots of warnings for comctl32/tests/listview.c when running in a virtual desktop

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

Bug ID: 36187
           Summary: valgrind shows lots of warnings for
                    comctl32/tests/listview.c when running in a virtual
                    desktop
           Product: Wine
           Version: 1.7.17
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: normal
          Priority: P2
         Component: comctl32
          Assignee: [e-mail]
          Reporter: [e-mail]

Created attachment 48319
  --> https://bugs.winehq.org/attachment.cgi?id=48319
valgrind log

==14980== Use of uninitialised value of size 4
==14980==    at 0x5666BD7: get_cached_glyph (graphics.c:587)
==14980==    by 0x5667351: render_string (graphics.c:762)
==14980==    by 0x5667816: dibdrv_ExtTextOut (graphics.c:860)
==14980==    by 0x5663F96: windrv_ExtTextOut (dc.c:738)
==14980==    by 0x569BD2E: ExtTextOutW (font.c:2447)
==14980==    by 0x526F318: DrawTextExW (text.c:1012)
==14980==    by 0x526F914: DrawTextW (text.c:1168)
==14980==    by 0x52543EC: NC_DrawCaption (nonclient.c:971)
==14980==    by 0x525478D: NC_DoNCPaint (nonclient.c:1055)
==14980==    by 0x5254B82: NC_HandleNCActivate (nonclient.c:1146)
==14980==    by 0x520424A: DEFWND_DefWinProc (defwnd.c:330)
==14980==    by 0x5205870: DefWindowProcA (defwnd.c:856)
==14980==    by 0x4CBBD1F: parent_wnd_proc (listview.c:525)
==14980==    by 0x528943D: ??? (winproc.c:173)
==14980==    by 0x52895B2: call_window_proc (winproc.c:244)
==14980==    by 0x528B74B: WINPROC_call_window (winproc.c:900)
==14980==    by 0x52480F8: call_window_proc (message.c:2223)
==14980==    by 0x524AF96: send_message (message.c:3260)
==14980==    by 0x524B5B4: SendMessageW (message.c:3454)
==14980==    by 0x521DA68: set_active_window (focus.c:153)
==14980==  Uninitialised value was created by a stack allocation
==14980==    at 0x5253FED: NC_DrawCaption (nonclient.c:906)
==14980==

==14980== Conditional jump or move depends on uninitialised value(s)
==14980==    at 0x56B7053: get_glyph_index_linked (freetype.c:7677)
==14980==    by 0x56B231C: get_glyph_outline (freetype.c:6246)
==14980==    by 0x56B6C11: freetype_GetGlyphOutline (freetype.c:7578)
==14980==    by 0x569CE8E: GetGlyphOutlineW (font.c:2821)
==14980==    by 0x5666F77: cache_glyph_bitmap (graphics.c:693)
==14980==    by 0x5667388: render_string (graphics.c:762)
==14980==    by 0x5667816: dibdrv_ExtTextOut (graphics.c:860)
==14980==    by 0x5663F96: windrv_ExtTextOut (dc.c:738)
==14980==    by 0x569BD2E: ExtTextOutW (font.c:2447)
==14980==    by 0x526F318: DrawTextExW (text.c:1012)
==14980==    by 0x526F914: DrawTextW (text.c:1168)
==14980==    by 0x52543EC: NC_DrawCaption (nonclient.c:971)
==14980==    by 0x525478D: NC_DoNCPaint (nonclient.c:1055)
==14980==    by 0x5254B82: NC_HandleNCActivate (nonclient.c:1146)
==14980==    by 0x520424A: DEFWND_DefWinProc (defwnd.c:330)
==14980==    by 0x5205870: DefWindowProcA (defwnd.c:856)
==14980==    by 0x4CBBD1F: parent_wnd_proc (listview.c:525)
==14980==    by 0x528943D: ??? (winproc.c:173)
==14980==    by 0x52895B2: call_window_proc (winproc.c:244)
==14980==    by 0x528B74B: WINPROC_call_window (winproc.c:900)
==14980==  Uninitialised value was created by a stack allocation
==14980==    at 0x5253FED: NC_DrawCaption (nonclient.c:906)
==14980==

==6389== Use of uninitialised value of size 4
==6389==    at 0x5F7798E: FT_Stream_ReadUShort (ftstream.c:514)
==6389==    by 0x5FB4AA6: tt_face_get_metrics (ttmtx.c:248)
==6389==    by 0x5F87790: tt_get_metrics (ttgload.c:78)
==6389==    by 0x5F8E35E: load_truetype_glyph (ttgload.c:1519)
==6389==    by 0x5F8F329: tt_glyph_load (ttgload.c:2402)
==6389==    by 0x5F7B4AE: FT_Load_Glyph (ftobjs.c:726)
==6389==    by 0x5FBBA78: af_loader_load_g (afloader.c:120)
==6389==    by 0x5FBECB1: af_autofitter_load_glyph (afloader.c:567)
==6389==    by 0x5F7B670: FT_Load_Glyph (ftobjs.c:717)
==6389==    by 0x5842BA6: get_glyph_outline (freetype.c:6395)
==6389==    by 0x58476B3: freetype_GetTextExtentExPoint (freetype.c:7815)
==6389==    by 0x5825FBF: get_char_positions (font.c:336)
==6389==    by 0x582830C: GetTextExtentExPointW (font.c:1330)
==6389==    by 0x53FE5F9: TEXT_NextLineW (text.c:693)
==6389==    by 0x53FF116: DrawTextExW (text.c:977)
==6389==    by 0x53FF914: DrawTextW (text.c:1168)
==6389==    by 0x53E43EC: NC_DrawCaption (nonclient.c:971)
==6389==    by 0x53E478D: NC_DoNCPaint (nonclient.c:1055)
==6389==    by 0x53E4B82: NC_HandleNCActivate (nonclient.c:1146)
==6389==    by 0x539424A: DEFWND_DefWinProc (defwnd.c:330)
==6389==  Uninitialised value was created by a stack allocation
==6389==    at 0x53E3FED: NC_DrawCaption (nonclient.c:906)
==6389==

etc. I'll attach the full log.

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/listview.c when running in a virtual desktop

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

Austin English  changed:

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

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/listview.c when running in a virtual desktop

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |valgrind

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/ listview.c when running in a virtual desktop

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

Nikolay Sivov  changed:

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

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/ listview.c when running in a virtual desktop

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

--- Comment #1 from Nikolay Sivov  ---
I sent a patch for this
https://www.winehq.org/pipermail/wine-patches/2017-January/156703.html. All of
those warnings seems to come from same place. There's one more fix to user32
part that I'll send later.

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/ listview.c when running in a virtual desktop

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

Nikolay Sivov  changed:

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

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/ listview.c when running in a virtual desktop

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

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |dabad0e98fb79e4da9fbd063a32
                   |                            |78255ecdb1242

--- Comment #2 from Nikolay Sivov  ---
Fixed with
http://source.winehq.org/git/wine.git/?a=commit;h=dabad0e98fb79e4da9fbd063a3....

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/ listview.c when running in a virtual desktop

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

Nikolay Sivov  changed:

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

[Bug 36187] valgrind shows lots of warnings for comctl32/tests/ listview.c when running in a virtual desktop

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

Alexandre Julliard  changed:

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

--- Comment #3 from Alexandre Julliard  ---
Closing bugs fixed in 2.0-rc5.