fix(wayland): release portal input capture when EI_EVENT_DISCONNECT is encountered, refactoring

See deskflow issue #8005.
This commit is contained in:
Dustin Kaiser
2025-04-17 21:57:47 +02:00
committed by Alban Browaeys
parent b1f2631eec
commit b7d1242c61
2 changed files with 16 additions and 9 deletions

View File

@@ -0,0 +1 @@
Fix InputLeap server inputs stuck when EIS reconnect to client after invalid EIS protocol.

View File

@@ -152,7 +152,7 @@ void EiScreen::cleanup_ei()
ei_device_set_user_data(ei_abs_, nullptr);
ei_abs_ = ei_device_unref(ei_abs_);
}
ei_seat_unref(ei_seat_);
ei_seat_ = ei_seat_unref(ei_seat_);
for (auto it = ei_devices_.begin(); it != ei_devices_.end(); it++) {
free(ei_device_get_user_data(*it));
ei_device_set_user_data(*it, nullptr);
@@ -738,7 +738,6 @@ void EiScreen::handle_portal_session_closed(const Event &event)
void EiScreen::handle_system_event(const Event& sysevent)
{
std::lock_guard<std::mutex> lock(mutex_);
bool disconnected = false;
// Only one ei_dispatch per system event, see the comment in
// EiEventQueueBuffer::addEvent
@@ -786,10 +785,20 @@ void EiScreen::handle_system_event(const Event& sysevent)
// We're using libei which emulates the various seat/device remove events
// so by the time we get here our EiScreen should be in a neutral state.
//
// We don't do anything here, we let the portal's Session.Closed signal
// handle the rest.
LOG_WARN("disconnected from EIS");
disconnected = true;
// We must release the xdg-portal InputCapture in case it is still active
// so that the cursor is usable and not stuck on the InputLeap server.
LOG_WARN("disconnected from eis, will afterwards commence attempt to reconnect");
if (is_primary_) {
LOG_DEBUG("re-allocating portal input capture connection and releasing active captures");
if (portal_input_capture_) {
if (portal_input_capture_->is_active()) {
portal_input_capture_->release();
}
delete portal_input_capture_;
portal_input_capture_ = new PortalInputCapture(this, this->events_);
}
}
this->handle_portal_session_closed(sysevent);
break;
case EI_EVENT_DEVICE_PAUSED:
LOG_DEBUG("device %s is paused", ei_device_get_name(device));
@@ -843,9 +852,6 @@ void EiScreen::handle_system_event(const Event& sysevent)
}
ei_event_unref(event);
}
if (disconnected)
ei_ = ei_unref(ei_);
}
void EiScreen::updateButtons()