From: Michael Stapelberg <michael@stapelberg.de>
Date: Mon, 12 Oct 2009 16:31:31 +0000 (+0200)
Subject: Bugfix: Fix NULL pointer dereference when reconfiguring screens
X-Git-Tag: 3.d-bf1~30
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=8f9e87124aa3ed50dcd297b82464ea3e98d5ac17;p=i3%2Fi3

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)
---

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;