summaryrefslogtreecommitdiffstats
path: root/binding/audiomixer.c
diff options
context:
space:
mode:
authorGeorge Kiagiadakis <george.kiagiadakis@collabora.com>2021-06-24 18:14:11 +0300
committerGeorge Kiagiadakis <george.kiagiadakis@collabora.com>2021-06-24 18:38:21 +0300
commitdadd3d771d5b27a455afffed8437c0a2e6db26b2 (patch)
treed577a336bac914df5f968828dbf1bdfe4b2f560c /binding/audiomixer.c
parentf1908e302b8c7317dc922164c37c4c50884684c6 (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
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