diff options
| author | Peter Hutterer <peter.hutterer@who-t.net> | 2018-08-08 15:22:41 +1000 |
|---|---|---|
| committer | Adam Jackson <ajax@redhat.com> | 2018-08-08 09:45:27 -0400 |
| commit | 4fe02b8da3ca58500f7e6e017e388907cf4a8b73 (patch) | |
| tree | ee260b96a0a5dbf562d3ae2694f7c670fa83b88a | |
| parent | c20e7b5e2222c0cae2a487264748fa5db711e6e4 (diff) | |
dix: check_modmap_change() returns Success, not true
Not sure what if anything calls XSetDeviceModifierMapping() but this would've
failed all the time. check_modmap_change() returns Success but we were
treating it like a boolean. Fix this.
Reported-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Keith Packard <keithp@keithp.com>
| -rw-r--r-- | dix/inpututils.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dix/inpututils.c b/dix/inpututils.c index 6bff9efab..c51c83547 100644 --- a/dix/inpututils.c +++ b/dix/inpututils.c @@ -282,7 +282,7 @@ change_modmap(ClientPtr client, DeviceIntPtr dev, KeyCode *modkeymap, else if (!IsFloating(dev) && GetMaster(dev, MASTER_KEYBOARD)->lastSlave == dev) { /* If this fails, expect the results to be weird. */ - if (check_modmap_change(client, dev->master, modmap)) + if (check_modmap_change(client, dev->master, modmap) == Success) do_modmap_change(client, dev->master, modmap); } |