diff options
author | Mike Gorse <mgorse@suse.com> | 2014-07-19 15:35:52 -0500 |
---|---|---|
committer | Mike Gorse <mgorse@suse.com> | 2014-07-19 15:35:52 -0500 |
commit | 4c0da161e7af297b34ca7fc31bd44c12baf78636 (patch) | |
tree | da462726b8ef218a8540930ffdffd7cdb683b45e /atspi/atspi-table-cell.c | |
parent | 925201dab4159ddce3ba29615223e6070123160a (diff) | |
download | at-spi2-core-4c0da161e7af297b34ca7fc31bd44c12baf78636.tar.gz |
Fix various memory leaks when checking D-Bus signatures
Diffstat (limited to 'atspi/atspi-table-cell.c')
-rw-r--r-- | atspi/atspi-table-cell.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/atspi/atspi-table-cell.c b/atspi/atspi-table-cell.c index 44ff4d03..f99ac854 100644 --- a/atspi/atspi-table-cell.c +++ b/atspi/atspi-table-cell.c @@ -179,6 +179,7 @@ atspi_table_cell_get_position (AtspiTableCell *obj, DBusMessage *reply; DBusMessageIter iter, iter_struct, iter_variant; dbus_int32_t d_row = -1, d_column = -1; + char *iter_sig; g_return_val_if_fail (obj != NULL, -1); @@ -193,9 +194,14 @@ atspi_table_cell_get_position (AtspiTableCell *obj, return FALSE; dbus_message_iter_recurse (&iter, &iter_variant); + iter_sig = dbus_message_iter_get_signature (&iter_variant); /* TODO: Also report error here */ - if (strcmp (dbus_message_iter_get_signature (&iter_variant), "(ii)") != 0) + if (strcmp (iter_sig, "(ii)") != 0) + { + dbus_free (iter_sig); return FALSE; + } + dbus_free (iter_sig); dbus_message_iter_recurse (&iter_variant, &iter_struct); dbus_message_iter_get_basic (&iter_struct, &d_row); |