From 6d6e2190801b278b3aab84d910737925b8df5c2f Mon Sep 17 00:00:00 2001 From: Simon Rozman Date: Thu, 27 Feb 2020 14:56:44 +0100 Subject: Reschedule EvaluateWintun between InstallInitialize and InstallFinalize WireGuard is always doing so-called "minor" upgrade. This makes it enough to run EvaluateWintun only once - somewhere after CostFinalize. The component state we bind our driver install/remove logic to will be a singleton: either be installed, left alone, or uninstalled. One single action for install product session and RemoveExistingProducts session. Other applications using Wintun might use the so-called "major" upgrade. In this scenario the existing product is completely uninstalled first. Including Wintun. The EvaluateWintun was called only once and it determined that Wintun driver should be uninstalled. Since the MSI did not execute EvaluateWintun again when installing the new product later, the Wintun remained uninstalled. In the case of major upgrades, the Wintun requires two separate action logics: what to do with Wintun on uninstall of the old product, and what to do with it when new product is installed. Therefore, EvaluateWintun has been moved between InstallInitialize and InstallFinalize to have MSI execute it in install and uninstall sessions. Reported-by: Dmitry Smirnov Tested-by: Simon Rozman Signed-off-by: Simon Rozman Signed-off-by: Jason A. Donenfeld --- installer/installer.wxs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/installer/installer.wxs b/installer/installer.wxs index 1883618..f1706cf 100644 --- a/installer/installer.wxs +++ b/installer/installer.wxs @@ -34,7 +34,7 @@ - + -- cgit v1.2.3-59-g8ed1b