new valgrind errors in gdiplus

[Dan Kegel]

Why this code, committed on the 21st, should only be
showing valgrind errors today, I don't know.
For a full and gory list of the new errors, see
http://kegel.com/wine/valgrind/logs-2008-06-30/vg-gdiplus_font-diff.txt

Here's a small excerpt:

+ Conditional jump or move depends on uninitialised value(s)
+at  CreateFontIndirectW (font.c:364)
+by  GdipCreateFontFamilyFromName (font.c:373)
+by  GdipGetGenericFontFamilySansSerif (font.c:516)
+by  test_getgenerics (font.c:206)
+by  func_font (font.c:253)
+by  run_test (test.h:449)
+by  main (test.h:498)
+  Uninitialised value was created by a stack allocation
+at  GdipCreateFontFamilyFromName (font.c:353)
...
+ Conditional jump or move depends on uninitialised value(s)
+at  fontcmp (freetype.c:3038)
+by  find_in_cache (freetype.c:3085)
+by  WineEngCreateFontInstance (freetype.c:3281)
+by  FONT_SelectObject (font.c:492)
+by  SelectObject (gdiobj.c:1129)
+by  GdipCreateFontFamilyFromName (font.c:374)
+by  GdipGetGenericFontFamilySansSerif (font.c:516)
+by  test_getgenerics (font.c:206)
+by  func_font (font.c:253)
+by  run_test (test.h:449)
+by  main (test.h:498)
+  Uninitialised value was created by a stack allocation
+at  GdipCreateFontFamilyFromName (font.c:353)
...
+ Conditional jump or move depends on uninitialised value(s)
+at  TranslateCharsetInfo (font.c:2577)
+by  WineEngCreateFontInstance (freetype.c:3319)
+by  FONT_SelectObject (font.c:492)
+by  SelectObject (gdiobj.c:1129)
+by  GdipCreateFontFamilyFromName (font.c:374)
+by  GdipGetGenericFontFamilySansSerif (font.c:516)
+by  test_getgenerics (font.c:206)
+by  func_font (font.c:253)
+by  run_test (test.h:449)
+by  main (test.h:498)
+  Uninitialised value was created by a stack allocation
+at  GdipCreateFontFamilyFromName (font.c:353)
...
+ Invalid free() / delete / delete[]
+at  notify_free (heap.c:199)
+by  RtlFreeHeap (heap.c:1284)
+by  GdipFree (gdiplus.c:122)
+by  GdipDeleteFontFamily (font.c:448)
+by  test_getgenerics (font.c:223)
+by  func_font (font.c:253)
+by  run_test (test.h:449)
+by  main (test.h:498)
+  Address 0x7f0162f0 is 0 bytes after a block of size 4,608 alloc'd
+at  notify_alloc (heap.c:191)
+by  RtlAllocateHeap (heap.c:1231)
+by  alloc_font (freetype.c:2840)
+by  WineEngCreateFontInstance (freetype.c:3301)
+by  FONT_SelectObject (font.c:492)
+by  SelectObject (gdiobj.c:1129)
+by  GdipCreateFontFromLogfontW (font.c:183)
+by  test_logfont (font.c:124)
+by  func_font (font.c:251)
+by  run_test (test.h:449)
+by  main (test.h:498)

Re: new valgrind errors in gdiplus

[Nikolay Sivov]

Dan Kegel wrote:
> Nikolai,
> your changes in
> http://source.winehq.org/git/wine.git/?a=commit;h=629761acde13283e0dfc6c42cab3e91e11fbbd88
> seem to have two valgrind errors,
> http://kegel.com/wine/valgrind/logs-2008-06-27/vg-gdiplus_graphicspath-diff.txt
>
> + Conditional jump or move depends on uninitialised value(s)
> +at  GdipAddPathPolygon (graphicspath.c:353)
> +by  test_polygon (graphicspath.c:608)
> +by  func_graphicspath (graphicspath.c:692)
> +by  run_test (test.h:449)
> +by  main (test.h:498)
> +  Uninitialised value was created by a stack allocation
> +at  test_polygon (graphicspath.c:589)
>
> ...
>
> + Conditional jump or move depends on uninitialised value(s)
> +at  GdipAddPathPolygon (graphicspath.c:353)
> +by  test_polygon (graphicspath.c:611)
> +by  func_graphicspath (graphicspath.c:692)
> +by  run_test (test.h:449)
> +by  main (test.h:498)
> +  Uninitialised value was created by a stack allocation
> +at  test_polygon (graphicspath.c:589)
>
> Could you have a look?
>   
Of course. I will.

new valgrind errors in gdiplus

[Dan Kegel]

Nikolai,
your changes in
http://source.winehq.org/git/wine.git/?a=commit;h=629761acde13283e0dfc6c42cab3e91e11fbbd88
seem to have two valgrind errors,
http://kegel.com/wine/valgrind/logs-2008-06-27/vg-gdiplus_graphicspath-diff.txt

+ Conditional jump or move depends on uninitialised value(s)
+at  GdipAddPathPolygon (graphicspath.c:353)
+by  test_polygon (graphicspath.c:608)
+by  func_graphicspath (graphicspath.c:692)
+by  run_test (test.h:449)
+by  main (test.h:498)
+  Uninitialised value was created by a stack allocation
+at  test_polygon (graphicspath.c:589)

...

+ Conditional jump or move depends on uninitialised value(s)
+at  GdipAddPathPolygon (graphicspath.c:353)
+by  test_polygon (graphicspath.c:611)
+by  func_graphicspath (graphicspath.c:692)
+by  run_test (test.h:449)
+by  main (test.h:498)
+  Uninitialised value was created by a stack allocation
+at  test_polygon (graphicspath.c:589)

Could you have a look?