mainnika/weston
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

exposay: Don't crash if no pointer is present

Browse Source 
If the compositor has never seen a mouse, exposay will crash because
the seat->pointer pointer is NULL.

Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
This commit is contained in:
Derek Foreman
2015-05-22 11:47:20 -05:00
committed by Pekka Paalanen
parent 35b7f25ae3
commit 3dd570e754
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
desktop-shell/exposay.c
+9 -6
View File
@@ -519,7 +519,8 @@ exposay_set_inactive(struct desktop_shell *shell)
struct weston_seat *seat = shell->exposay.seat; struct weston_seat *seat = shell->exposay.seat;
weston_keyboard_end_grab(seat->keyboard); weston_keyboard_end_grab(seat->keyboard);
weston_pointer_end_grab(seat->pointer); if (seat->pointer_device_count)
weston_pointer_end_grab(seat->pointer);
if (seat->keyboard->input_method_resource) if (seat->keyboard->input_method_resource)
seat->keyboard->grab = &seat->keyboard->input_method_grab; seat->keyboard->grab = &seat->keyboard->input_method_grab;
@@ -570,11 +571,13 @@ exposay_transition_active(struct desktop_shell *shell)
weston_keyboard_set_focus(seat->keyboard, NULL); weston_keyboard_set_focus(seat->keyboard, NULL);
shell->exposay.grab_ptr.interface = &exposay_ptr_grab; shell->exposay.grab_ptr.interface = &exposay_ptr_grab;
weston_pointer_start_grab(seat->pointer, if (seat->pointer_device_count) {
&shell->exposay.grab_ptr); weston_pointer_start_grab(seat->pointer,
weston_pointer_set_focus(seat->pointer, NULL, &shell->exposay.grab_ptr);
seat->pointer->x, seat->pointer->y); weston_pointer_set_focus(seat->pointer, NULL,
seat->pointer->x,
seat->pointer->y);
}
wl_list_for_each(shell_output, &shell->output_list, link) { wl_list_for_each(shell_output, &shell->output_list, link) {
enum exposay_layout_state state; enum exposay_layout_state state;