http://bugs.winehq.org/show_bug.cgi?id=11784
Alex Villacís Lasso a_villacis@palosanto.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |a_villacis@palosanto.com
--- Comment #2 from Alex Villacís Lasso a_villacis@palosanto.com 2008-04-04 15:34:32 --- (In reply to comment #1)
WndClsEx.hIcon = (HICON)(LoadImage(0, "c:\windows\temp\Failed.gif", IMAGE_BITMAP, 0, 0, LR_LOADFROMFILE));
On windows this test doesn't crash, but in wine it does. I'm not exactly sure now where to fix this bug :(. Any help appreciated.
So what is the expected result in Windows? Should LoadImage actually support GIF (and PNG and JPG and WMF and ...) formats? Or should it just refuse to load anything and return a null handle (or something else), instead of crashing? GIF (and other) support could be ripped from dlls/oleaut32/olepicture.c, *if and only if* LoadImage is demonstrated to support GIF (and other) formats. Tests for olepicture could be adapted for LoadImage to test this. If, on the other hand, LoadImage should refuse to load GIFs without crashing, it should have some validation instead, placed in dlls/user32/cursoricon.c .
In any case, relevant tests should be placed in dlls/user32/tests/cursoricon.c along with the rest of tests.