[PATCH] msi: Fix a leak on error path (Coverity).

Sven Baars sven.wine at gmail.com
Thu Sep 26 09:32:28 CDT 2019


Signed-off-by: Sven Baars <sven.wine at gmail.com>
---
 dlls/msi/dialog.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c
index 6ccf46ec30..8a28e4d4af 100644
--- a/dlls/msi/dialog.c
+++ b/dlls/msi/dialog.c
@@ -4033,7 +4033,11 @@ static msi_dialog *dialog_create( MSIPACKAGE *package, const WCHAR *name, msi_di
     msiobj_release( &rec->hdr );
 
     rec = MSI_CreateRecord(2);
-    if (!rec) return NULL;
+    if (!rec)
+    {
+        msi_dialog_destroy(dialog);
+        return NULL;
+    }
     MSI_RecordSetStringW(rec, 1, name);
     MSI_RecordSetStringW(rec, 2, szDialogCreated);
     MSI_ProcessMessage(package, INSTALLMESSAGE_ACTIONSTART, rec);
-- 
2.17.1




More information about the wine-devel mailing list