From 0d8a826ef54f831fcbacd9204587d2e50a37732b Mon Sep 17 00:00:00 2001
From: Hans Leidekker <hans@codeweavers.com>
Date: Wed, 6 May 2009 15:57:22 +0200
Subject: [PATCH] msi: Reject NULL or empty patch package parameter in
 MsiApplyPatch.

---
 dlls/msi/msi.c           |  3 +++
 dlls/msi/tests/package.c | 12 ++++++++++++
 2 files changed, 15 insertions(+)

diff --git a/dlls/msi/msi.c b/dlls/msi/msi.c
index 58a1856f229..ed8b4ab24de 100644
--- a/dlls/msi/msi.c
+++ b/dlls/msi/msi.c
@@ -315,6 +315,9 @@ static UINT MSI_ApplyPatchW(LPCWSTR szPatchPackage, LPCWSTR szProductCode, LPCWS
     static const WCHAR patcheq[] = {'P','A','T','C','H','=',0};
     static WCHAR empty[] = {0};
 
+    if (!szPatchPackage || !szPatchPackage[0])
+        return ERROR_INVALID_PARAMETER;
+
     if (!szProductCode)
     {
         r = MsiOpenDatabaseW(szPatchPackage, MSIDBOPEN_READONLY, &patch);
diff --git a/dlls/msi/tests/package.c b/dlls/msi/tests/package.c
index a7e52614fdf..308a9ccd142 100644
--- a/dlls/msi/tests/package.c
+++ b/dlls/msi/tests/package.c
@@ -11604,6 +11604,17 @@ static void test_MsiApplyMultiplePatches(void)
     todo_wine ok(r == ERROR_FILE_NOT_FOUND, "Expected ERROR_FILE_NOT_FOUND, got %u\n", r);
 }
 
+static void test_MsiApplyPatch(void)
+{
+    UINT r;
+
+    r = MsiApplyPatch(NULL, NULL, INSTALLTYPE_DEFAULT, NULL);
+    ok(r == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %u\n", r);
+
+    r = MsiApplyPatch("", NULL, INSTALLTYPE_DEFAULT, NULL);
+    ok(r == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %u\n", r);
+}
+
 START_TEST(package)
 {
     HMODULE hmsi = GetModuleHandleA("msi.dll");
@@ -11642,4 +11653,5 @@ START_TEST(package)
     test_MsiGetProductProperty();
     test_MsiSetProperty();
     test_MsiApplyMultiplePatches();
+    test_MsiApplyPatch();
 }