From 1ffbae54a7212b90f480c73c39539314cbb47bdb Mon Sep 17 00:00:00 2001 From: Yaroslav Chvanov Date: Wed, 8 Mar 2023 19:34:28 +0300 Subject: [PATCH] ratbagd: change type of a variable to fix stack smashing sd-bus requires an int variable to be passed to SD_BUS_TYPE_BOOLEAN calls, see man sd_bus_message_read (3). We have had an issue of the same sort recently in 77fcaf6d281788edc3d52e52a19964bfc20786bc which was in some freshly new code, but this time it's something we have been shipping for a while. --- ratbagd/ratbagd-profile.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ratbagd/ratbagd-profile.c b/ratbagd/ratbagd-profile.c index 5d081ae3..08416b3b 100644 --- a/ratbagd/ratbagd-profile.c +++ b/ratbagd/ratbagd-profile.c @@ -330,7 +330,7 @@ ratbagd_profile_set_disabled(sd_bus *bus, { struct ratbagd_profile *profile = userdata; int r; - bool disabled; + int disabled; CHECK_CALL(sd_bus_message_read(m, "b", &disabled));