From 8f9e87124aa3ed50dcd297b82464ea3e98d5ac17 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 354d0fa6..7e08725b 100644 --- a/src/handlers.c +++ b/src/handlers.c @@ -931,7 +931,7 @@ int handle_expose_event(void *data, xcb_connection_t *conn, xcb_expose_event_t * else { uint32_t background_color; /* Distinguish if the window is currently focused… */ - if (CUR_CELL->currently_focused == client) + 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;