summaryrefslogtreecommitdiffstats
path: root/binding/audiomixer.c
diff options
context:
space:
mode:
authorGeorge Kiagiadakis <george.kiagiadakis@collabora.com>2021-06-24 18:14:11 +0300
committerJan-Simon Moeller <jsmoeller@linuxfoundation.org>2021-06-28 18:27:47 +0000
commitadcee8121ca25a14ee4cc50d15211b075b769ab8 (patch)
tree48ba6e4100c42db942c91ecec77c116cdbb665f7 /binding/audiomixer.c
parent5b1a74b1ea6cfc49e20300bdf5b3806bcb41343d (diff)
* 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.c4
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