From 9c6839969cd77b8fbb5064a5913a22f6d66d0fbe Mon Sep 17 00:00:00 2001
From: alkl58 <alkl58@protonmail.com>
Date: Thu, 9 Mar 2023 13:05:19 +0100
Subject: [PATCH] Check path validity on Windows will validate NTFS permissions
 for network drives. Signed-off-by: Alkl58 <alkl58@protonmail.com>

---
 src/common/utility.h       |  2 ++
 src/common/utility_win.cpp | 10 ++++++++++
 src/gui/folderman.cpp      | 12 +++++++++++-
 3 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/src/common/utility.h b/src/common/utility.h
index e19f0da6ae1e..9f5f71a5b976 100644
--- a/src/common/utility.h
+++ b/src/common/utility.h
@@ -275,6 +275,8 @@ namespace Utility {
 
     OCSYNC_EXPORT QString formatWinError(long error);
 
+    OCSYNC_EXPORT bool canCreateFileInPath(const QString &path);
+
     class OCSYNC_EXPORT NtfsPermissionLookupRAII
     {
     public:
diff --git a/src/common/utility_win.cpp b/src/common/utility_win.cpp
index 37b348660c0d..c9b1b1641da4 100644
--- a/src/common/utility_win.cpp
+++ b/src/common/utility_win.cpp
@@ -33,6 +33,7 @@
 #include <QFile>
 #include <QLibrary>
 #include <QSettings>
+#include <QTemporaryFile>
 
 extern Q_CORE_EXPORT int qt_ntfs_permission_lookup;
 
@@ -430,6 +431,15 @@ void Utility::UnixTimeToLargeIntegerFiletime(time_t t, LARGE_INTEGER *hundredNSe
     hundredNSecs->HighPart = ll >>32;
 }
 
+bool Utility::canCreateFileInPath(const QString &path)
+{
+    Q_ASSERT(!path.isEmpty());
+    const auto pathWithSlash = !path.endsWith(QLatin1Char('/'))
+        ? path + QLatin1Char('/')
+        : path;
+    QTemporaryFile testFile(pathWithSlash + QStringLiteral("~$write-test-file-XXXXXX"));
+    return testFile.open();
+}
 
 QString Utility::formatWinError(long errorCode)
 {
diff --git a/src/gui/folderman.cpp b/src/gui/folderman.cpp
index 1dfc68503883..708b81358024 100644
--- a/src/gui/folderman.cpp
+++ b/src/gui/folderman.cpp
@@ -1701,10 +1701,20 @@ static QString checkPathValidityRecursive(const QString &path)
         return FolderMan::tr("The selected path is not a folder!");
     }
 
+    #ifdef Q_OS_WIN
+    if (!selFile.isWritable()) {
+        // isWritable() doesn't cover all NTFS permissions
+        // try creating and removing a test file, and make sure it is excluded from sync
+        if (!Utility::canCreateFileInPath(path)) {
+            return FolderMan::tr("You have no permission to write to the selected folder!");
+        }
+    }
+    #else
     if (!selFile.isWritable()) {
         return FolderMan::tr("You have no permission to write to the selected folder!");
     }
-    return QString();
+    #endif
+    return {};
 }
 
 // QFileInfo::canonicalPath returns an empty string if the file does not exist.