diff options
| author | Peter Wu <peter@lekensteyn.nl> | 2014-11-06 18:29:07 +0100 |
|---|---|---|
| committer | Peter Wu <peter@lekensteyn.nl> | 2014-11-18 10:30:39 +0100 |
| commit | 4618c07a3d235053e25e6f0362625bf8ebb0e820 (patch) | |
| tree | 7156d11fcb3b42cc65a217538ab9eb3b4f764f6d | |
| parent | c9daf7b69e06533012408b03ab1bed1507ab3ea2 (diff) | |
| download | upower-4618c07a3d235053e25e6f0362625bf8ebb0e820.tar.gz | |
linux: fix memleak when reading critical action
g_variant_get for string types must be freed as documented at
https://developer.gnome.org/glib/stable/gvariant-format-strings.html#gvariant-format-strings-strings
Since we just want to compare it, use a pointer as documented at
https://developer.gnome.org/glib/stable/gvariant-format-strings.html#gvariant-format-strings-pointers
Signed-off-by: Peter Wu <peter@lekensteyn.nl>
| -rw-r--r-- | src/linux/up-backend.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/linux/up-backend.c b/src/linux/up-backend.c index f588e74..f45ce29 100644 --- a/src/linux/up-backend.c +++ b/src/linux/up-backend.c @@ -346,7 +346,7 @@ check_action_result (GVariant *result) if (result) { const char *s; - g_variant_get (result, "(s)", &s); + g_variant_get (result, "(&s)", &s); if (g_strcmp0 (s, "yes") == 0) return TRUE; } |