From 8c79f8dc155ba696572be30ee8c2ddfc577ae418 Mon Sep 17 00:00:00 2001 From: Ben Collerson Date: Tue, 10 Oct 2023 16:42:46 +1000 Subject: [PATCH] check client is mapped before setting border color For some reason brave configured for as a wayland client triggers this code on startup and segfaults. Checking if the client is mapped fixes this, like with the previous fix for urgent border colour. References: 887fde65a3010905aa10f373cfcfe540cfc1781e Fixes: 72a7d78a1a7926a207539eb50f44b2e1eb089d49 --- dwl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dwl.c b/dwl.c index 4cfa4db..a7d41b0 100644 --- a/dwl.c +++ b/dwl.c @@ -2593,7 +2593,8 @@ urgent(struct wl_listener *listener, void *data) if (!c || c == focustop(selmon)) return; - client_set_border_color(c, urgentcolor); + if (client_is_mapped(c)) + client_set_border_color(c, urgentcolor); c->isurgent = 1; printstatus(); }