From 9e89a8ed3c54ad4c9ad9cc939c995a7196037889 Mon Sep 17 00:00:00 2001 From: Simon McVittie Date: Wed, 16 Jan 2013 17:15:01 +0000 Subject: Only ignore passwords in gnome-keyring if Empathy has copied them When passwords are stored in the Parameters by MC and no SASLAuthentication handler is used, e.g. under Sugar, we currently save passwords into gnome-keyring, if enabled at compile time; so, we want to read them back out of gnome-keyring. This conflicts slightly with wanting to ignore passwords in gnome-keyring that are left over from an Empathy 3.0 schema migration that failed due to fd.o #42088. To avoid failure to authenticate/re-prompt if the password has been changed on the service side, ignore passwords that are going to be deleted; but to avoid breaking Sugar, don't ignore any other passwords we might find in gnome-keyring. Signed-off-by: Simon McVittie Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59468 --- src/mcd-account-manager-default.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/mcd-account-manager-default.c b/src/mcd-account-manager-default.c index 8d05fb8c..ef39ba16 100644 --- a/src/mcd-account-manager-default.c +++ b/src/mcd-account-manager-default.c @@ -335,14 +335,14 @@ _get_secrets_from_keyring (const McpAccountStorage *self, "account", account, "param", "password", NULL); + + /* behave as if it had already been deleted, i.e. we never + * actually found it... */ + param = NULL; + value = NULL; } gnome_keyring_found_list_free (empathy_items); - - /* behave as if it had already been deleted, i.e. we never - * actually found it... */ - param = NULL; - value = NULL; } if (param != NULL && value != NULL) -- cgit v1.2.1