diff options
author | Pavel Holejsovsky <pavel.holejsovsky@gmail.com> | 2012-03-31 19:12:48 +0200 |
---|---|---|
committer | Pavel Holejsovsky <pholejs@src.gnome.org> | 2012-04-11 20:38:53 +0200 |
commit | 2bcf6395a17c655d5164b1a9fb3c3bb5aedda376 (patch) | |
tree | 05850eb4ccb381773cc6381d6243a290446aab07 | |
parent | 041aeabea9d84376230354938d1205616e4a560a (diff) | |
download | gobject-introspection-2bcf6395a17c655d5164b1a9fb3c3bb5aedda376.tar.gz |
Avoid _get_func() consuming reference from input info
Fix of leak in 4c9424e18d71237f438a99bc5f2d45ae7de60b78 was a bit
overaggressive, stealing also one reference from input 'info'
argument.
Also fixes another bug in that commit - local 'func' shadowing the
return value, causing that function always returned NULL even when
some result was actually found.
https://bugzilla.gnome.org/show_bug.cgi?id=673282
-rw-r--r-- | girepository/giobjectinfo.c | 3 | ||||
-rw-r--r-- | tests/repository/gitypelibtest.c | 13 |
2 files changed, 14 insertions, 2 deletions
diff --git a/girepository/giobjectinfo.c b/girepository/giobjectinfo.c index 3b8f64bb..7ba2a93b 100644 --- a/girepository/giobjectinfo.c +++ b/girepository/giobjectinfo.c @@ -819,7 +819,7 @@ _get_func(GIObjectInfo *info, GIObjectInfo *parent_info; gpointer func = NULL; - parent_info = info; + parent_info = g_base_info_ref (info); while (parent_info != NULL) { parents = g_slist_prepend (parents, parent_info); @@ -828,7 +828,6 @@ _get_func(GIObjectInfo *info, for (l = parents; l; l = l->next) { - GIObjectInfoRefFunction func; parent_info = l->data; symbol = getter (parent_info); if (symbol == NULL) diff --git a/tests/repository/gitypelibtest.c b/tests/repository/gitypelibtest.c index 15722a6d..c53eab29 100644 --- a/tests/repository/gitypelibtest.c +++ b/tests/repository/gitypelibtest.c @@ -156,6 +156,18 @@ test_is_pointer_for_struct_arg (GIRepository *repo) g_base_info_unref (variant_info); } +static void +test_fundamental_get_ref_function_pointer (GIRepository *repo) +{ + GIObjectInfo *info; + + g_assert (g_irepository_require (repo, "Regress", NULL, 0, NULL)); + info = g_irepository_find_by_name (repo, "Regress", + "TestFundamentalObject"); + g_object_info_get_ref_function_pointer (info); + g_base_info_unref (info); +} + int main(int argc, char **argv) { @@ -170,6 +182,7 @@ main(int argc, char **argv) test_enum_and_flags_static_methods (repo); test_size_of_gvalue (repo); test_is_pointer_for_struct_arg (repo); + test_fundamental_get_ref_function_pointer (repo); exit(0); } |