keystore: try memory keystore on the first try
authorThomas Guillem <thomas@gllm.fr>
Tue, 12 Feb 2019 14:25:27 +0000 (15:25 +0100)
committerThomas Guillem <thomas@gllm.fr>
Tue, 12 Feb 2019 14:46:53 +0000 (15:46 +0100)
This will speed up remote and protected accesses opening when the user already
entered a credential. Indeed, accessess generally call vlc_credential_get()
with NULL dialog title/fmt arguments first in order to get the password saved
from url/vlc-option but without triggering a dialog. Then if it is failing,
they call the same function with valid dialog arguments in order to trigger the
keystore and the dialog.

This commits will trigger the memory keystore from the first try. This will
prevent accesses to always try the default ("Guest") account if the user
already provided a valid credential.

src/misc/keystore.c

index 62d1822..53edb35 100644 (file)
@@ -440,9 +440,6 @@ vlc_credential_get(vlc_credential *p_credential, vlc_object_t *p_parent,
 
         case GET_FROM_MEMORY_KEYSTORE:
         {
-            if (!psz_dialog_title || !psz_dialog_fmt)
-                return false;
-
             vlc_keystore *p_keystore = get_memory_keystore(p_parent);
             if (p_keystore != NULL)
                 credential_find_keystore(p_credential, p_keystore);