common: Fix handling of redirectdrives setting
authorMartin Fleisz <martin.fleisz@thincast.com>
Tue, 1 Oct 2019 13:38:31 +0000 (15:38 +0200)
committerakallabeth <akallabeth@users.noreply.github.com>
Tue, 1 Oct 2019 13:56:22 +0000 (15:56 +0200)
client/common/cmdline.c

index c23c4d5..abb883a 100644 (file)
@@ -3300,15 +3300,7 @@ BOOL freerdp_client_load_addins(rdpChannels* channels, rdpSettings* settings)
 
                settings->DeviceRedirection = TRUE;
        }
-
-       if (settings->RedirectDrives || settings->RedirectHomeDrive
-           || settings->RedirectSerialPorts
-           || settings->RedirectSmartCards || settings->RedirectPrinters)
-       {
-               settings->DeviceRedirection = TRUE; /* All of these features require rdpdr */
-       }
-
-       if (settings->RedirectDrives)
+       else if (settings->RedirectDrives)
        {
                if (!freerdp_device_collection_find(settings, "drive"))
                {
@@ -3322,6 +3314,13 @@ BOOL freerdp_client_load_addins(rdpChannels* channels, rdpSettings* settings)
                }
        }
 
+       if (settings->RedirectDrives || settings->RedirectHomeDrive
+           || settings->RedirectSerialPorts
+           || settings->RedirectSmartCards || settings->RedirectPrinters)
+       {
+               settings->DeviceRedirection = TRUE; /* All of these features require rdpdr */
+       }
+
        if (settings->RedirectHomeDrive)
        {
                if (!freerdp_device_collection_find(settings, "drive"))