diff options
author | Guillaume Desmottes <guillaume.desmottes@collabora.co.uk> | 2013-09-23 16:24:17 +0200 |
---|---|---|
committer | Guillaume Desmottes <guillaume.desmottes@collabora.co.uk> | 2013-09-25 11:00:48 +0200 |
commit | b4e924942d3090ad9df1d0443bfa98cccaf6680e (patch) | |
tree | 7ebfa6a6efd99a64a168ce75eee6d6ed6823cc87 /telepathy-logger/log-store-xml.c | |
parent | 0f3c1d3f1df18d6d6d1533cc1d4656ff64607fb0 (diff) | |
download | telepathy-logger-b4e924942d3090ad9df1d0443bfa98cccaf6680e.tar.gz |
stop using tp_account_manager_get_valid_accounts()
The code example in log-walker.c was wrong as get_valid_accounts() is
(transfer container). dup_valid_accounts() is (transfer full) so it's correct
now.
https://bugs.freedesktop.org/show_bug.cgi?id=69797
Diffstat (limited to 'telepathy-logger/log-store-xml.c')
-rw-r--r-- | telepathy-logger/log-store-xml.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/telepathy-logger/log-store-xml.c b/telepathy-logger/log-store-xml.c index 03f5d3c..ec291b4 100644 --- a/telepathy-logger/log-store-xml.c +++ b/telepathy-logger/log-store-xml.c @@ -1020,7 +1020,7 @@ log_store_xml_search_hit_new (TplLogStoreXml *self, /* FIXME: This assumes the account manager is prepared, but the * synchronous API forces this. See bug #599189. */ - accounts = tp_account_manager_get_valid_accounts ( + accounts = tp_account_manager_dup_valid_accounts ( self->priv->account_manager); for (l = accounts; l != NULL && account == NULL; l = g_list_next (l)) @@ -1033,7 +1033,7 @@ log_store_xml_search_hit_new (TplLogStoreXml *self, account = acc; g_free (name); } - g_list_free (accounts); + g_list_free_full (accounts, g_object_unref); if (is_chatroom) target = tpl_entity_new_from_room_id (chat_id); |