[vlc-commits] [Git][videolan/vlc][master] Revert "smb2: fix anonymous login"

[Thomas Guillem (@tguillem)]

Thomas Guillem pushed to branch master at VideoLAN / VLC


Commits:
06692b49 by Thomas Guillem at 2022-07-07T10:45:42+00:00
Revert "smb2: fix anonymous login"

This reverts commit 205963ad09401ab3cbe5bf92c7f9b109092f87f4.

According to git blame on libsmb2, anonymous login was always enabled
with a NULL password.

I don't know what happened when I tested this reverted commit, I may
have mix up VLC or/and smb/smb2/dsm builds.

For future reference, see libsmb2/lib/ntlmssp.c:
...
encode_ntlm_auth(...)
{
...
    if (auth_data->password == NULL) {
            anonymous = 1;
            goto encode;
    }
...
}

Fixes #27113

- - - - -


1 changed file:

- modules/access/smb2.c


Changes:

=====================================
modules/access/smb2.c
=====================================
@@ -687,8 +687,8 @@ vlc_smb2_connect_open_share(stream_t *access, const char *url,
     if (!username)
     {
         username = "Guest";
-        /* An empty password enable ntlmssp anonymous login */
-        password = "";
+        /* A NULL password enable ntlmssp anonymous login */
+        password = NULL;
     }
 
     struct vlc_access_cache_entry *cache_entry =



View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/06692b49ceda37d05fb3e7764efbecf731a11773

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/06692b49ceda37d05fb3e7764efbecf731a11773
You're receiving this email because of your account on code.videolan.org.


VideoLAN code repository instance