drath/libratbag
1
0
Fork 1
mirror of https://github.com/libratbag/libratbag.git synced 2026-11-07 23:23:52 +01:00
Code Issues Projects Releases Wiki Activity

ratbagd: fix stack smashing

Browse Source 
A regression I introduced while modifying the disableable resolutions
patch.
It went unnoticed, because it only crashes with optimizations enabled.
In my defense, this exception in D-Bus API that boolean messages must
give address of an int and not a bool is a weird peculiarity...
This commit is contained in:
Yaroslav Chvanov
2022-12-26 23:40:37 +03:00
parent fdb4d65aa4
commit 77fcaf6d28
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
ratbagd/ratbagd-resolution.c
View File

@@ -196,12 +196,12 @@ static int ratbagd_resolution_set_disabled(sd_bus *bus,
sd_bus_error *error)
{
struct ratbagd_resolution *resolution = userdata;
bool is_disabled;
int is_disabled;
int r;
CHECK_CALL(sd_bus_message_read(m, "b", &is_disabled));
r = ratbag_resolution_set_disabled(resolution->lib_resolution, is_disabled);
r = ratbag_resolution_set_disabled(resolution->lib_resolution, !!is_disabled);
if (r == 0) {
sd_bus *bus = sd_bus_message_get_bus(m);
sd_bus_emit_properties_changed(bus,