From 6190f0c7bf3b598e766c67b5be88aee220d8c80f Mon Sep 17 00:00:00 2001 From: Benjamin Tissoires Date: Mon, 14 Mar 2016 16:19:15 +0100 Subject: [PATCH] Make sure we do not put random values in the .macro field of buttons Otherwise, as soon as we set an action for a button, we get a nice invalid free on cleanup. Signed-off-by: Benjamin Tissoires --- src/libratbag.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/libratbag.c b/src/libratbag.c index b62038cb..98cadb1e 100644 --- a/src/libratbag.c +++ b/src/libratbag.c @@ -959,7 +959,7 @@ ratbag_button_get_button(struct ratbag_button *button) LIBRATBAG_EXPORT enum ratbag_error_code ratbag_button_set_button(struct ratbag_button *button, unsigned int btn) { - struct ratbag_button_action action; + struct ratbag_button_action action = {0}; int rc; if (!button->profile->device->driver->write_button) @@ -989,7 +989,7 @@ LIBRATBAG_EXPORT enum ratbag_error_code ratbag_button_set_special(struct ratbag_button *button, enum ratbag_button_action_special act) { - struct ratbag_button_action action; + struct ratbag_button_action action = {0}; int rc; /* FIXME: range checks */ @@ -1027,7 +1027,7 @@ ratbag_button_set_key(struct ratbag_button *button, unsigned int *modifiers, size_t sz) { - struct ratbag_button_action action; + struct ratbag_button_action action = {0}; int rc; /* FIXME: range checks */