projects
/
platform
/
upstream
/
at-spi2-core.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
[prevent][73691] Fix for resource leak
[platform/upstream/at-spi2-core.git]
/
test
/
at_spi2_tool.c
diff --git
a/test/at_spi2_tool.c
b/test/at_spi2_tool.c
index
63a3adb
..
cca84f1
100644
(file)
--- a/
test/at_spi2_tool.c
+++ b/
test/at_spi2_tool.c
@@
-633,7
+633,7
@@
static void _atspi_tree_traverse(const char *app_name, bool dump, bool check, bo
if (!dump && !check)
printf("%s\n", name);
if (!dump && !check)
printf("%s\n", name);
- if ((check || dump) && name && !strcmp(name, app_name)) {
+ if ((check || dump) && name &&
app_name &&
!strcmp(name, app_name)) {
app_name_matched = true;
_print_module_legend();
app_name_matched = true;
_print_module_legend();
@@
-655,9
+655,8
@@
static void _atspi_tree_traverse(const char *app_name, bool dump, bool check, bo
} else {
printf("\n");
}
} else {
printf("\n");
}
-
- free(name);
}
}
+ free(name);
}
if (!app_name_matched && (dump || check))
}
if (!app_name_matched && (dump || check))
@@
-670,6
+669,7
@@
static void _at_spi_client_enable(gboolean enabled)
{
static GDBusProxy *proxy = NULL; //we keep proxy (dbus connection) until program exits
GVariant *result;
{
static GDBusProxy *proxy = NULL; //we keep proxy (dbus connection) until program exits
GVariant *result;
+ GVariant *enabled_variant;
GError *error = NULL;
GDBusProxyFlags flags = G_DBUS_PROXY_FLAGS_NONE;
GError *error = NULL;
GDBusProxyFlags flags = G_DBUS_PROXY_FLAGS_NONE;
@@
-690,13
+690,16
@@
static void _at_spi_client_enable(gboolean enabled)
}
}
}
}
+ enabled_variant = g_variant_new_boolean(enabled);
result = g_dbus_proxy_call_sync(proxy,
"Set",
result = g_dbus_proxy_call_sync(proxy,
"Set",
- g_variant_new ("(ssv)", "org.a11y.Status", "IsEnabled",
g_variant_new_boolean(enabled)
),
+ g_variant_new ("(ssv)", "org.a11y.Status", "IsEnabled",
enabled_variant
),
G_DBUS_CALL_FLAGS_NONE,
-1,
NULL,
&error);
G_DBUS_CALL_FLAGS_NONE,
-1,
NULL,
&error);
+ if (enabled_variant)
+ g_variant_unref(enabled_variant);
if (result)
g_variant_unref(result);
if (result)
g_variant_unref(result);