Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Commit

Permalink
Don't pass NULL surfaces to wlr focus functions (#8)
Browse files Browse the repository at this point in the history
Turns out that this will hit asserts that will abort dwl.
  • Loading branch information
Shugyousha authored Jun 14, 2020
1 parent df10c47 commit 63f09d1
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions dwl.c
Original file line number Diff line number Diff line change
Expand Up @@ -595,9 +595,11 @@ focusclient(Client *c, struct wlr_surface *surface, int lift)
* If the focused surface has changed, tell the seat to have the
* keyboard enter the new surface. wlroots will keep track of this and
* automatically send key events to the appropriate clients. If surface
* is NULL, this will clear focus.
* is NULL, we clear the focus instead.
*/
if (surface != psurface) {
if (!surface) {
wlr_seat_pointer_notify_clear_focus(seat);
} else if (surface != psurface) {
kb = wlr_seat_get_keyboard(seat);
wlr_seat_keyboard_notify_enter(seat, surface,
kb->keycodes, kb->num_keycodes, &kb->modifiers);
Expand Down Expand Up @@ -897,11 +899,17 @@ pointerfocus(Client *c, struct wlr_surface *surface, double sx, double sy,
wlr_seat_pointer_notify_motion(seat, time, sx, sy);
return;
}

/* If surface is NULL, clear pointer focus, otherwise let the client
* know that the mouse cursor has entered one of its surfaces. */
if (!surface) {
wlr_seat_pointer_notify_clear_focus(seat);
return;
}

wlr_seat_pointer_notify_enter(seat, surface, sx, sy);
/* If keyboard focus follows mouse, enforce that */
if (sloppyfocus && surface)
if (sloppyfocus)
focusclient(c, surface, 0);
}

Expand Down

0 comments on commit 63f09d1

Please sign in to comment.