input/seat: Reset command handler context in seat_destroy()

This fixes a dangling reference which causes a use-after-free.
This commit is contained in:
Andri Yngvason 2020-12-28 21:36:12 +00:00 committed by Simon Ser
parent e11bc058ac
commit 9ef5cc03f1

View File

@ -15,6 +15,7 @@
#include "config.h" #include "config.h"
#include "list.h" #include "list.h"
#include "log.h" #include "log.h"
#include "sway/config.h"
#include "sway/desktop.h" #include "sway/desktop.h"
#include "sway/input/cursor.h" #include "sway/input/cursor.h"
#include "sway/input/input-manager.h" #include "sway/input/input-manager.h"
@ -53,6 +54,9 @@ static void seat_node_destroy(struct sway_seat_node *seat_node) {
} }
void seat_destroy(struct sway_seat *seat) { void seat_destroy(struct sway_seat *seat) {
if (seat == config->handler_context.seat) {
config->handler_context.seat = input_manager_get_default_seat();
}
struct sway_seat_device *seat_device, *next; struct sway_seat_device *seat_device, *next;
wl_list_for_each_safe(seat_device, next, &seat->devices, link) { wl_list_for_each_safe(seat_device, next, &seat->devices, link) {
seat_device_destroy(seat_device); seat_device_destroy(seat_device);