[gnome-control-center] power: Use g_variant_get_string() instead of g_variant_get()



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]