[PATCH libinput 2/3] touchpad: remove the lid switch listener on device_removed
Peter Hutterer
peter.hutterer at who-t.net
Fri May 5 08:22:02 UTC 2017
Sequence triggered by the xorg driver, but basically: if the touchpad is
destroyed before the lid switch, the event listener wasn't removed and an
assertion was triggered.
Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
src/evdev-mt-touchpad.c | 6 ++++++
test/test-lid.c | 26 ++++++++++++++++++++++++++
2 files changed, 32 insertions(+)
diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c
index 8140c446..e0757e17 100644
--- a/src/evdev-mt-touchpad.c
+++ b/src/evdev-mt-touchpad.c
@@ -1768,6 +1768,12 @@ tp_interface_device_removed(struct evdev_device *device,
tp->dwt.keyboard = NULL;
}
+ if (removed_device == tp->lid_switch.lid_switch) {
+ libinput_device_remove_event_listener(
+ &tp->lid_switch.lid_switch_listener);
+ tp->lid_switch.lid_switch = NULL;
+ }
+
if (tp->sendevents.current_mode !=
LIBINPUT_CONFIG_SEND_EVENTS_DISABLED_ON_EXTERNAL_MOUSE)
return;
diff --git a/test/test-lid.c b/test/test-lid.c
index a5f439a0..6b2fa3c5 100644
--- a/test/test-lid.c
+++ b/test/test-lid.c
@@ -407,6 +407,30 @@ START_TEST(lid_open_on_key_touchpad_enabled)
}
END_TEST
+START_TEST(lid_suspend_with_touchpad)
+{
+ struct libinput *li;
+ struct litest_device *touchpad, *sw;
+
+ li = litest_create_context();
+
+ sw = litest_add_device(li, LITEST_LID_SWITCH);
+ litest_drain_events(li);
+
+ touchpad = litest_add_device(li, LITEST_SYNAPTICS_I2C);
+ litest_delete_device(touchpad);
+ touchpad = litest_add_device(li, LITEST_SYNAPTICS_I2C);
+ litest_drain_events(li);
+
+ litest_delete_device(sw);
+ litest_drain_events(li);
+ litest_delete_device(touchpad);
+ litest_drain_events(li);
+
+ libinput_unref(li);
+}
+END_TEST
+
START_TEST(lid_update_hw_on_key)
{
struct litest_device *sw = litest_current_device();
@@ -469,5 +493,7 @@ litest_setup_tests_lid(void)
litest_add("lid:keyboard", lid_open_on_key, LITEST_SWITCH, LITEST_ANY);
litest_add("lid:keyboard", lid_open_on_key_touchpad_enabled, LITEST_SWITCH, LITEST_ANY);
+ litest_add_no_device("lid:disable_touchpad", lid_suspend_with_touchpad);
+
litest_add_for_device("lid:buggy", lid_update_hw_on_key, LITEST_LID_SWITCH_SURFACE3);
}
--
2.12.2
More information about the wayland-devel
mailing list