diff options
| author | Xavier Claessens <xavier.claessens@collabora.co.uk> | 2012-07-09 12:35:01 +0200 |
|---|---|---|
| committer | Xavier Claessens <xavier.claessens@collabora.co.uk> | 2012-07-11 12:09:34 +0200 |
| commit | 0daa22ca3858138aee10f8bf46a07fd15c1fcf49 (patch) | |
| tree | 7d538d74bde0832f3c0dbfa4c484ffe094af3fbb | |
| parent | 150c89d45c6b7d5a2d10bc36cff5689ea1ca72d8 (diff) | |
| download | telepathy-mission-control-0daa22ca3858138aee10f8bf46a07fd15c1fcf49.tar.gz | |
McdAccountManagerDefault: Fix a possible double free
If the account is already in the hashtable, g_hash_table_insert()
will set @removed as value, but free it since the key already in
the table is kept.
https://bugs.freedesktop.org/show_bug.cgi?id=51842
| -rw-r--r-- | src/mcd-account-manager-default.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mcd-account-manager-default.c b/src/mcd-account-manager-default.c index deae1630..e156de1b 100644 --- a/src/mcd-account-manager-default.c +++ b/src/mcd-account-manager-default.c @@ -77,7 +77,7 @@ _delete_from_keyring (const McpAccountStorage *self, { /* flag the whole account as purged */ gchar *removed = g_strdup (account); - g_hash_table_insert (amd->removed_accounts, removed, removed); + g_hash_table_replace (amd->removed_accounts, removed, removed); } else { |