CURSORICON_SimulateLoadingFromResourceW param check. VFP6 EXEs now work.

Uwe Bonnes bon at elektron.ikp.physik.tu-darmstadt.de
Tue Mar 23 14:29:10 CST 2004


>>>>> "C" == C Daniel Mojoli B <cdmojoli at idea.com.py> writes:

    C> Uwe Bonnes wrote:
    >>>>>>> "C" == C Daniel Mojoli B <cdmojoli at idea.com.py> writes:
    >>>>>>> 
    >>>>>>> 
    >>
    C> Add minimal parameter validation to function
    C> CURSORICON_SimulateLoadingFromResourceW(...). We check that the icon
    >> Providing a test case for such special behaviour is a good idea...
    >> 
    >> 
    C> I created this patch as a result of consulting I performed at a
    C> customer's site. I no longer have access to the winedbg info I had

It this what you meant?

The test succeeds on NT and crashes on wine
-- 
Uwe Bonnes                bon at elektron.ikp.physik.tu-darmstadt.de

Institut fuer Kernphysik  Schlossgartenstrasse 9  64289 Darmstadt
--------- Tel. 06151 162516 -------- Fax. 06151 164321 ----------
Index: wine/dlls/user/tests/win.c
===================================================================
RCS file: /home/wine/wine/dlls/user/tests/win.c,v
retrieving revision 1.18
diff -u -r1.18 win.c
--- wine/dlls/user/tests/win.c	19 Mar 2004 19:14:45 -0000	1.18
+++ wine/dlls/user/tests/win.c	23 Mar 2004 20:25:10 -0000
@@ -1222,6 +1222,15 @@
     DestroyMenu(hMenu);
 }
 
+void test_LoadIcon()
+{
+    HANDLE hLI;
+
+    /* Loading a random file should fail, not crash/*
+    hLI =LoadImageA(NULL,"win.c",IMAGE_CURSOR,16,16,LR_DEFAULTCOLOR|LR_LOADFROMFILE);
+    ok (hLI == 0,"LoadImage on random file shouldn't succeed\n");
+}
+
 START_TEST(win)
 {
     pGetAncestor = (void *)GetProcAddress( GetModuleHandleA("user32.dll"), "GetAncestor" );
@@ -1249,6 +1258,7 @@
 
     test_mdi();
     test_icons();
+    test_LoadIcon();
     test_SetWindowPos(hwndMain);
     test_SetMenu(hwndMain);
 



More information about the wine-patches mailing list