MSI: don't pass NULL

Saulius Krasuckas saulius2 at ar.fi.lt
Sun Sep 4 08:22:53 CDT 2005


Log message:
	Saulius Krasuckas <saulius.krasuckas at ieee.org>
	Older platforms choke on a NULL as the third parameter to MsiViewGetError.


--- dlls/msi/tests/db.c	2005-08-24 17:44:46.000000000 +0200
+++ /mnt/vcd2/wine/dlls/msi/tests/db.c	2005-09-04 14:39:00.000000000 +0200
@@ -388,7 +388,9 @@ static void test_viewmodify(void)
     ok(r == ERROR_SUCCESS, "query failed\n");
 
     /* check what the error function reports without doing anything */
-    r = MsiViewGetError( 0, NULL, NULL );
+    sz = 0;
+    /* passing NULL as the 3rd param make function to crash on older platforms */
+    r = MsiViewGetError( 0, NULL, &sz );
     ok(r == MSIDBERROR_INVALIDARG, "MsiViewGetError return\n");
 
     /* open a view */



More information about the wine-patches mailing list