diff options
author | George Kiagiadakis <george.kiagiadakis@collabora.com> | 2021-06-24 18:14:11 +0300 |
---|---|---|
committer | George Kiagiadakis <george.kiagiadakis@collabora.com> | 2021-06-24 18:38:21 +0300 |
commit | dadd3d771d5b27a455afffed8437c0a2e6db26b2 (patch) | |
tree | d577a336bac914df5f968828dbf1bdfe4b2f560c | |
parent | f1908e302b8c7317dc922164c37c4c50884684c6 (diff) |
audiomixer: fix calls to g_variant_lookup()marlin_12.93.0marlin_12.91.0marlin_12.90.1marlin_12.90.0marlin/12.93.0marlin/12.91.0marlin/12.90.1marlin/12.90.012.93.012.91.012.90.112.90.0
* actually get values instead of storing them into intermediate
pointers on the stack
* fix crash on 32-bit architectures
* no need to use NULL sentinel
Bug-AGL: SPEC-3976
Signed-off-by: George Kiagiadakis <george.kiagiadakis@collabora.com>
Change-Id: I88647ae21405fe3bd7e57c82d3e953be2767c0c9
-rw-r--r-- | binding/audiomixer.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/binding/audiomixer.c b/binding/audiomixer.c index 78dd347..97ad622 100644 --- a/binding/audiomixer.c +++ b/binding/audiomixer.c @@ -56,8 +56,8 @@ get_mixer_controls (struct audiomixer * self, guint32 node_id, gdouble * vol, gb g_autoptr (GVariant) v = NULL; g_signal_emit_by_name (self->mixer_api, "get-volume", node_id, &v); return v && - g_variant_lookup (v, "volume", "d", &vol, NULL) && - g_variant_lookup (v, "mute", "b", &mute, NULL); + g_variant_lookup (v, "volume", "d", vol) && + g_variant_lookup (v, "mute", "b", mute); } static void |