New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

input/seat: handle wlr_seat destroy #8034

Draft

emersion wants to merge 1 commit into swaywm:master from emersion:seat-handle-destroy

Member

emersion commented Mar 4, 2024 •

edited

Doesn't work properly: IPC stuff is cleaned up in a wl_display destroy listener, and wlr_backend destroy happens after and tries to send IPC events…

Also if input_manager_get_default_seat() is called it might re-create the default seat after it's destroyed…


          input/seat: handle wlr_seat destroy

b5fd7de

emersion added a commit to emersion/sway that referenced this pull request


          server: fix wlr_seat use-after-free on exit

165068f

Same as [1].

I originally tried to properly handle seat destruction, but that
turned out to be a can of worms [2].

[1]: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4590
[2]: swaywm#8034

emersion mentioned this pull request

server: fix wlr_seat use-after-free on exit #8035

Merged

emersion added a commit that referenced this pull request


          server: fix wlr_seat use-after-free on exit

2b08e79

Same as [1].

I originally tried to properly handle seat destruction, but that
turned out to be a can of worms [2].

[1]: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4590
[2]: #8034

frosklis pushed a commit to frosklis/sway-frosklis that referenced this pull request


          server: fix wlr_seat use-after-free on exit

656b8e3

Same as [1].

I originally tried to properly handle seat destruction, but that
turned out to be a can of worms [2].

[1]: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4590
[2]: swaywm#8034

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment