From 76f97ff01babdcadcb63e71b7ae6caaa1f80928a Mon Sep 17 00:00:00 2001
From: Frans van Dorsselaer <17404029+dorssel@users.noreply.github.com>
Date: Sat, 20 Jan 2024 20:04:07 +0100
Subject: [PATCH] Cleanup

---
 Installer/PowerShell.wxs |  5 +----
 Installer/Product.wxs    | 14 --------------
 Installer/Server.wxs     |  4 ++--
 Installer/WSL.wxs        |  9 +++------
 Usbipd/NativeMethods.txt |  3 ---
 5 files changed, 6 insertions(+), 29 deletions(-)

diff --git a/Installer/PowerShell.wxs b/Installer/PowerShell.wxs
index 636d959..4d13285 100644
--- a/Installer/PowerShell.wxs
+++ b/Installer/PowerShell.wxs
@@ -4,10 +4,7 @@ SPDX-FileCopyrightText: 2024 Frans van Dorsselaer
 
 SPDX-License-Identifier: GPL-3.0-only
 -->
-<Wix
-    xmlns="http://wixtoolset.org/schemas/v4/wxs"
-    xmlns:fw="http://wixtoolset.org/schemas/v4/wxs/firewall"
-    >
+<Wix xmlns="http://wixtoolset.org/schemas/v4/wxs">
     <Fragment>
         <DirectoryRef Id="APPLICATIONFOLDER">
             <Directory Id="PowerShell" Name="PowerShell" />
diff --git a/Installer/Product.wxs b/Installer/Product.wxs
index 79e288f..ecbbd4b 100644
--- a/Installer/Product.wxs
+++ b/Installer/Product.wxs
@@ -41,20 +41,6 @@ SPDX-License-Identifier: GPL-3.0-only
 
         <Property Id="MSIDEPLOYMENTCOMPLIANT" Value="1" />
 
-        <!--
-        We use .NET trimmed publishing, which will modify the .NET DLLs, but keeps the file version the same.
-        Therefore, we must always overwrite all files, as the trimming may be different from one build to another.
-        This is safe, since we only install files in our own dedicated application directory.
-        -->
-        <SetProperty
-            Action="SetREINSTALLMODE"
-            Id="REINSTALLMODE"
-            Value="amus"
-            Before="CostInitialize"
-            Sequence="first"
-            Condition="NOT REINSTALLMODE"
-            />
-
         <!--
         MSI specifies that TARGETDIR overrides the default installation location, and winget relies on that.
         WiX, however, uses APPLICATIONFOLDER.
diff --git a/Installer/Server.wxs b/Installer/Server.wxs
index 7ce680a..4e9eb2e 100644
--- a/Installer/Server.wxs
+++ b/Installer/Server.wxs
@@ -81,7 +81,7 @@ SPDX-License-Identifier: GPL-3.0-only
 
         <!-- This will restore the original Windows drivers for devices that were forced bound. -->
         <SetProperty Id="RestoreDrivers" Value="&quot;[#usbipd.exe]&quot; unbind --all" Sequence="execute" Before="RestoreDrivers" />
-        <CustomAction Id="RestoreDrivers" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec64" Return="ignore" Impersonate="no" Execute="deferred" />
+        <CustomAction Id="RestoreDrivers" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec" Return="ignore" Impersonate="no" Execute="deferred" />
 
         <!--
         This will *try to* start the service. However, unlike ServiceControl, it will not fail if it can't.
@@ -89,7 +89,7 @@ SPDX-License-Identifier: GPL-3.0-only
         The CLI tool will inform the user if the service is not running and that a reboot should fix that.
         -->
         <SetProperty Id="TryStartService" Value="&quot;[System64Folder]\sc.exe&quot; start usbipd" Sequence="execute" Before="TryStartService" />
-        <CustomAction Id="TryStartService" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec64" Return="ignore" Impersonate="no" Execute="deferred" />
+        <CustomAction Id="TryStartService" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec" Return="ignore" Impersonate="no" Execute="deferred" />
 
         <InstallExecuteSequence>
             <!-- Condition: usbipd.exe is installed and will be uninstalled -->
diff --git a/Installer/WSL.wxs b/Installer/WSL.wxs
index 59a7762..d3e0bb6 100644
--- a/Installer/WSL.wxs
+++ b/Installer/WSL.wxs
@@ -4,10 +4,7 @@ SPDX-FileCopyrightText: 2024 Frans van Dorsselaer
 
 SPDX-License-Identifier: GPL-3.0-only
 -->
-<Wix
-    xmlns="http://wixtoolset.org/schemas/v4/wxs"
-    xmlns:fw="http://wixtoolset.org/schemas/v4/wxs/firewall"
-    >
+<Wix xmlns="http://wixtoolset.org/schemas/v4/wxs">
     <Fragment>
         <DirectoryRef Id="APPLICATIONFOLDER">
             <!-- Windows uses uppercase spelling for folders related to WSL (and lowercase for files). -->
@@ -36,7 +33,7 @@ SPDX-License-Identifier: GPL-3.0-only
         -->
         <!--
         <SetProperty Id="FixExecutionRights_file.id" Value="&quot;[System64Folder]\icacls.exe&quot; &quot;[#file.id]&quot; /deny &quot;*S-1-1-0:(X)&quot;" Sequence="execute" Before="FixExecutionRights_file.id" />
-        <CustomAction Id="FixExecutionRights_file.id" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec64" Return="ignore" Impersonate="no" Execute="deferred" />
+        <CustomAction Id="FixExecutionRights_file.id" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec" Return="ignore" Impersonate="no" Execute="deferred" />
 
         <InstallExecuteSequence>
             <Custom Action="FixExecutionRights_file.id" Before="InstallFinalize" Condition="$file.id=3" />
@@ -44,7 +41,7 @@ SPDX-License-Identifier: GPL-3.0-only
         -->
 
         <SetProperty Id="FixExecutionRights_WSL_README.md" Value="&quot;[System64Folder]\icacls.exe&quot; &quot;[#WSL_README.md]&quot; /deny &quot;*S-1-1-0:(X)&quot;" Sequence="execute" Before="FixExecutionRights_WSL_README.md" />
-        <CustomAction Id="FixExecutionRights_WSL_README.md" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec64" Return="ignore" Impersonate="no" Execute="deferred" />
+        <CustomAction Id="FixExecutionRights_WSL_README.md" BinaryRef="Wix4UtilCA_X64" DllEntry="WixQuietExec" Return="ignore" Impersonate="no" Execute="deferred" />
 
         <InstallExecuteSequence>
             <Custom Action="FixExecutionRights_WSL_README.md" After="InstallFiles" Condition="$WSL_README.md=3" />
diff --git a/Usbipd/NativeMethods.txt b/Usbipd/NativeMethods.txt
index aea583c..41e37a0 100644
--- a/Usbipd/NativeMethods.txt
+++ b/Usbipd/NativeMethods.txt
@@ -28,7 +28,6 @@ CreateFile
 DeviceIoControl
 DiInstallDevice
 GetCurrentProcess
-GetTokenInformation
 LookupPrivilegeValue
 OpenProcessToken
 RegDeleteKey
@@ -112,5 +111,3 @@ IOCTL_USB_HUB_CYCLE_PORT
 // USB_NODE_CONNECTION_INFORMATION_EX
 USB_NODE_CONNECTION_INFORMATION_EX_V2
 USB_CYCLE_PORT_PARAMS
-
-tcp_keepalive