[PATCH 5/6] msi: Don't queue a parent dialog to be shown in EndDialog.

Zebediah Figura z.figura12 at gmail.com
Thu Jul 20 23:43:14 CDT 2017


When using "Return", the parent dialog already exists, so there is
no need to do this. Moreover, if the parent dialog is queued, a
subsequent "Exit" would not properly exit the parent dialog.

Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
---
 dlls/msi/dialog.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c
index 6b57606..a5b839d 100644
--- a/dlls/msi/dialog.c
+++ b/dlls/msi/dialog.c
@@ -4381,12 +4381,7 @@ static UINT event_end_dialog( msi_dialog *dialog, const WCHAR *argument )
     else if (!strcmpW( argument, ignoreW ))
         dialog->package->CurrentInstallState = ERROR_SUCCESS;
     else if (!strcmpW( argument, returnW ))
-    {
-        msi_dialog *parent = dialog->parent;
-        msi_free( dialog->package->next_dialog );
-        dialog->package->next_dialog = (parent) ? strdupW( parent->name ) : NULL;
         dialog->package->CurrentInstallState = ERROR_SUCCESS;
-    }
     else
     {
         ERR("Unknown argument string %s\n", debugstr_w(argument));
-- 
2.7.4




More information about the wine-patches mailing list