diff options
author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2013-01-16 17:15:01 +0000 |
---|---|---|
committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2013-05-01 19:51:25 +0100 |
commit | c9ea70be25c04011cf8fe7dd8b554dba4392057b (patch) | |
tree | 2cb7e4267f56607ed01a3c02baecda3fc948b130 | |
parent | ef2ea07d1b27dabd87e0ae028396138f78d1970a (diff) | |
download | telepathy-mission-control-c9ea70be25c04011cf8fe7dd8b554dba4392057b.tar.gz |
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 <simon.mcvittie@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59468
-rw-r--r-- | src/mcd-account-manager-default.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/mcd-account-manager-default.c b/src/mcd-account-manager-default.c index 2fb03000..e72f60b0 100644 --- a/src/mcd-account-manager-default.c +++ b/src/mcd-account-manager-default.c @@ -331,14 +331,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) |