summaryrefslogtreecommitdiff
path: root/telepathy-logger/log-store-xml.c
diff options
context:
space:
mode:
authorGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>2013-09-23 16:24:17 +0200
committerGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>2013-09-25 11:00:48 +0200
commitb4e924942d3090ad9df1d0443bfa98cccaf6680e (patch)
tree7ebfa6a6efd99a64a168ce75eee6d6ed6823cc87 /telepathy-logger/log-store-xml.c
parent0f3c1d3f1df18d6d6d1533cc1d4656ff64607fb0 (diff)
downloadtelepathy-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.c4
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);