[vlc-commits] keystore: accept empty username

Thomas Guillem git at videolan.org
Wed Sep 2 11:15:43 CEST 2020


vlc | branch: master | Thomas Guillem <thomas at gllm.fr> | Tue Sep  1 10:57:29 2020 +0200| [384526b41285662944625bdd0b61c03868358279] | committer: Thomas Guillem

keystore: accept empty username

The http protocol accepts empty username. Other protocols may not accept it.
This will result on a fail of the login function, leaving the possibility to
try a new login.

Fixes #25074

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=384526b41285662944625bdd0b61c03868358279
---

 src/misc/keystore.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/misc/keystore.c b/src/misc/keystore.c
index b5b4cc7e50..d58e8dea98 100644
--- a/src/misc/keystore.c
+++ b/src/misc/keystore.c
@@ -196,8 +196,7 @@ find_closest_path(vlc_keystore_entry *p_entries, unsigned i_count,
 static bool
 is_credential_valid(vlc_credential *p_credential)
 {
-    if (p_credential->psz_username && *p_credential->psz_username != '\0'
-     && p_credential->psz_password)
+    if (p_credential->psz_username && p_credential->psz_password)
         return true;
     p_credential->psz_password = NULL;
     return false;



More information about the vlc-commits mailing list