commit c06a180dd574c24ca50766461c10d3a6bfa3ad2b
author Boyan Ding <stu_dby@126.com> Sun Jul 06 11:44:58 2014 +0800
committer Pekka Paalanen <pekka.paalanen@collabora.co.uk> Tue Aug 19 16:24:58 2014 +0300
tree b6ac986fa8bfb83d895c45a7935405569baedcfe
parent a0fae4657fab4f1708449d5ee5e1b92e5e90c0aa

xwm: check whether the picked seat can be NULL

The seat picked in weston_wm_window_handle_moveresize can sometimes
be NULL when it is (somehow) triggered with all buttons released.

This patch checks whether the seat is NULL to avoid NULL dereference.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=80837
Signed-off-by: Boyan Ding <stu_dby@126.com>

xwayland/window-manager.c

diff --git a/xwayland/window-manager.c b/xwayland/window-manager.c
index 6cac278..f1523f5 100644
--- a/xwayland/window-manager.c
+++ b/xwayland/window-manager.c
@@ -1244,8 +1244,8 @@
 	struct weston_shell_interface *shell_interface =
 		&wm->server->compositor->shell_interface;
 
-	if (seat->pointer->button_count != 1 || !window->view
-	    || seat->pointer->focus != window->view)
+	if (seat == NULL || seat->pointer->button_count != 1
+	    || !window->view || seat->pointer->focus != window->view)
 		return;
 
 	detail = client_message->data.data32[2];