[gnome-control-center] power: Use g_variant_get_string() instead of g_variant_get()
- From: Mario Sanchez Prada <msanchez src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-control-center] power: Use g_variant_get_string() instead of g_variant_get()
- Date: Mon, 27 Apr 2015 14:56:55 +0000 (UTC)
commit 9ff192ab2ed599ce619701d65a39f0b94aac5185
Author: Mario Sanchez Prada <mario endlessm com>
Date: Mon Apr 27 11:38:15 2015 +0100
power: Use g_variant_get_string() instead of g_variant_get()
Fixes a memory leak as the "(s)" format string was being used, but no
call to g_free() was happening afterwards.
https://bugzilla.gnome.org/show_bug.cgi?id=748411
panels/power/cc-power-panel.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
---
diff --git a/panels/power/cc-power-panel.c b/panels/power/cc-power-panel.c
index 487aef4..6d040a8 100644
--- a/panels/power/cc-power-panel.c
+++ b/panels/power/cc-power-panel.c
@@ -1935,9 +1935,11 @@ add_automatic_suspend_section (CcPowerPanel *self)
if (result)
{
- g_variant_get (result, "(s)", &s);
- if (g_strcmp0 (s, "yes") == 0)
+ GVariant *result_variant = g_variant_get_child_value (result, 0);
+ if (g_strcmp0 (g_variant_get_string (result_variant, NULL), "yes") == 0)
value = 1;
+
+ g_variant_unref(result_variant);
g_variant_unref(result);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]