diff options
author | George Kiagiadakis <george.kiagiadakis@collabora.com> | 2021-06-24 18:14:11 +0300 |
---|---|---|
committer | Jan-Simon Moeller <jsmoeller@linuxfoundation.org> | 2021-06-28 18:27:47 +0000 |
commit | adcee8121ca25a14ee4cc50d15211b075b769ab8 (patch) | |
tree | 48ba6e4100c42db942c91ecec77c116cdbb665f7 /binding/audiomixer.c | |
parent | 5b1a74b1ea6cfc49e20300bdf5b3806bcb41343d (diff) |
audiomixer: fix calls to g_variant_lookup()koi_11.0.5koi_11.0.4koi_11.0.3koi/11.0.5koi/11.0.4koi/11.0.311.0.511.0.411.0.3koi
* 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
(cherry picked from commit dadd3d771d5b27a455afffed8437c0a2e6db26b2)
Diffstat (limited to 'binding/audiomixer.c')
-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 |