[Windows] Fix: enables passwordless SMB authentication

thexai · Apr 25, 2021 · 93c33d2 · 93c33d2
1 parent 27ed641
commit 93c33d2
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/xbmc/platform/win32/filesystem/Win32SMBDirectory.cpp b/xbmc/platform/win32/filesystem/Win32SMBDirectory.cpp
@@ -723,10 +723,20 @@ bool CWin32SMBDirectory::ConnectAndAuthenticate(CURL& url, bool allowPromptForCr
   connInfo.dwType = RESOURCETYPE_ANY;
   connInfo.lpRemoteName = (LPWSTR)serverShareNameW.c_str();
   DWORD connRes;
+
   for (int i = 0; i < 3; i++) // make up to three attempts to connect
   {
+    // try with provided user/password or NULL user/password (NULL = current Windows user/password)
     connRes = WNetAddConnection2W(&connInfo, passwordW.empty() ? NULL : (LPWSTR)passwordW.c_str(),
                                   usernameW.empty() ? NULL : (LPWSTR)usernameW.c_str(), CONNECT_TEMPORARY);
+
+    // try with passwordless access (guest user and no password)
+    if (usernameW.empty() && passwordW.empty() &&
+        (connRes == ERROR_ACCESS_DENIED || connRes == ERROR_BAD_USERNAME ||
+         connRes == ERROR_INVALID_PASSWORD || connRes == ERROR_LOGON_FAILURE ||
+         connRes == ERROR_LOGON_TYPE_NOT_GRANTED || connRes == ERROR_LOGON_NOT_GRANTED))
+      connRes = WNetAddConnection2W(&connInfo, L"", L"guest", CONNECT_TEMPORARY);
+
     if (connRes == NO_ERROR)
     {
       CLog::LogF(LOGDEBUG, "Connected to \"%s\" %s", serverShareName.c_str(), loginDescr.c_str());