From: Michael Stapelberg <michael@stapelberg.de>
Date: Fri, 28 Sep 2012 16:23:01 +0000 (+0200)
Subject: Bugfix: don’t crash when dragged floating window closes (Thanks darkraven)
X-Git-Tag: 4.4~99^2^2
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=8480b4cb58c30a5fa4ff9761aaed198efa8429f8;p=i3%2Fi3

Bugfix: don’t crash when dragged floating window closes (Thanks darkraven)

Previously, while the loop would not be executed anymore, the callback
(for actually repositioning the window after a MotionNotify) would still
get called once. This commit avoids that, thus fixing the crash.

fixes #819
---

diff --git a/src/floating.c b/src/floating.c
index 3d2c1d31..5df39bf1 100644
--- a/src/floating.c
+++ b/src/floating.c
@@ -570,7 +570,7 @@ void drag_pointer(Con *con, const xcb_button_press_event_t *event, xcb_window_t
                 free(inside_event);
         } while ((inside_event = xcb_poll_for_event(conn)) != NULL);
 
-        if (last_motion_notify == NULL)
+        if (last_motion_notify == NULL || loop_done)
             continue;
 
         new_x = ((xcb_motion_notify_event_t*)last_motion_notify)->root_x;