diff options
author | Cosimo Cecchi <cosimoc@gnome.org> | 2012-09-13 13:37:50 -0400 |
---|---|---|
committer | Cosimo Cecchi <cosimoc@gnome.org> | 2012-09-13 16:38:32 -0400 |
commit | d1e52eb847dcabf7a2be07dbd71cfcb4e0805306 (patch) | |
tree | 5d72c24541f5c413a8639448e5d53eaa0f8669e4 /atk-adaptor/accessible-cache.c | |
parent | 3bdee8357cd975dfb03ea98a3bdbfcfc3c1da7e0 (diff) | |
download | at-spi2-core-d1e52eb847dcabf7a2be07dbd71cfcb4e0805306.tar.gz |
atk-adaptor: plug some refcount leaks
cache->add_traversal holds GObject references, so when we remove/pop
items from it, we should also drop the reference, unless it's transfered
to another queue.
https://bugzilla.gnome.org/show_bug.cgi?id=683979
Diffstat (limited to 'atk-adaptor/accessible-cache.c')
-rw-r--r-- | atk-adaptor/accessible-cache.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/atk-adaptor/accessible-cache.c b/atk-adaptor/accessible-cache.c index afaedb0f..3a30b514 100644 --- a/atk-adaptor/accessible-cache.c +++ b/atk-adaptor/accessible-cache.c @@ -180,7 +180,10 @@ remove_object (GObject * source, GObject * gobj, gpointer data) g_hash_table_remove (cache->objects, gobj); } else - g_queue_remove (cache->add_traversal, gobj); + { + g_queue_remove (cache->add_traversal, gobj); + g_object_unref (gobj); + } } static void @@ -277,12 +280,14 @@ add_pending_items (gpointer data) while (!g_queue_is_empty (cache->add_traversal)) { AtkStateSet *set; - + + /* cache->add_traversal holds a ref to current */ current = g_queue_pop_head (cache->add_traversal); set = atk_object_ref_state_set (current); if (set && !atk_state_set_contains_state (set, ATK_STATE_TRANSIENT)) { + /* transfer the ref into to_add */ g_queue_push_tail (to_add, current); if (!spi_cache_in (cache, G_OBJECT (current)) && !atk_state_set_contains_state (set, ATK_STATE_MANAGES_DESCENDANTS) && @@ -291,6 +296,11 @@ add_pending_items (gpointer data) append_children (current, cache->add_traversal); } } + else + { + /* drop the ref for the removed object */ + g_object_unref (current); + } if (set) g_object_unref (set); |