From 04884ab8cdc2cb0d17fc5bfdd13ad0660074dc40 Mon Sep 17 00:00:00 2001 From: Michael Stapelberg Date: Mon, 12 Oct 2009 18:31:31 +0200 Subject: [PATCH] Bugfix: Fix NULL pointer dereference when reconfiguring screens This did not happen all the time. It seems like you need to have a container which is in stacking/tabbing mode on the screen which is being reconfigured. (when doing xrandr --output VGA1 --off for example) --- src/handlers.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/handlers.c b/src/handlers.c index 5b456fb7..d4e04b68 100644 --- a/src/handlers.c +++ b/src/handlers.c @@ -719,7 +719,7 @@ int handle_expose_event(void *data, xcb_connection_t *conn, xcb_expose_event_t * if (client->urgent) background_color = config.client.urgent.background; /* Distinguish if the window is currently focused… */ - else if (CUR_CELL->currently_focused == client) + else if (CUR_CELL != NULL && CUR_CELL->currently_focused == client) background_color = config.client.focused.background; /* …or if it is the focused window in a not focused container */ else background_color = config.client.focused_inactive.background; -- 2.39.5