From 6cdf8a901a80845d21f64cb8ea8f7952ac2e268a Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Thu, 31 Aug 2017 08:18:42 +1000 Subject: [PATCH] ratbagd: fix use of potentially unused variable Simply abort if the macro value is bad, that's a libratbag bug and should never happen. Found in #312 Signed-off-by: Peter Hutterer --- ratbagd/ratbagd-button.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ratbagd/ratbagd-button.c b/ratbagd/ratbagd-button.c index ebc434a4..40b9a7ac 100644 --- a/ratbagd/ratbagd-button.c +++ b/ratbagd/ratbagd-button.c @@ -305,6 +305,8 @@ static int ratbagd_button_get_macro(sd_bus *bus, case RATBAG_MACRO_EVENT_WAIT: value = ratbag_button_macro_get_event_timeout(macro, idx); break; + default: + abort(); } r = sd_bus_message_append(reply, "(uu)", type, value);