[PATCH xwayland 2/3] Check for frame being NULL before setting/unsetting flags

Axel Davy davy at clipper.ens.fr
Fri Nov 15 02:25:50 PST 2013


This patch (Again, I had the same), solves the last bugs for XWayland,
but for this one, I'm not sure it is the correct fix.

In fact it would mean than focus_window would be an unmapped windows,
which looks strange to me.

Axel Davy

Le 15/11/2013 11:01, Dima Ryazanov a écrit :
> Fixes a crash in Firefox when clicking an "install plugin" popup.
>
> Signed-off-by: Dima Ryazanov <dima at gmail.com>
> ---
>   src/xwayland/window-manager.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/xwayland/window-manager.c b/src/xwayland/window-manager.c
> index 5ee9480..6d29026 100644
> --- a/src/xwayland/window-manager.c
> +++ b/src/xwayland/window-manager.c
> @@ -695,12 +695,14 @@ weston_wm_window_activate(struct wl_listener *listener, void *data)
>   	}
>   
>   	if (wm->focus_window) {
> -		frame_unset_flag(wm->focus_window->frame, FRAME_FLAG_ACTIVE);
> +		if (wm->focus_window->frame)
> +			frame_unset_flag(wm->focus_window->frame, FRAME_FLAG_ACTIVE);
>   		weston_wm_window_schedule_repaint(wm->focus_window);
>   	}
>   	wm->focus_window = window;
>   	if (wm->focus_window) {
> -		frame_set_flag(wm->focus_window->frame, FRAME_FLAG_ACTIVE);
> +		if (wm->focus_window->frame)
> +			frame_set_flag(wm->focus_window->frame, FRAME_FLAG_ACTIVE);
>   		weston_wm_window_schedule_repaint(wm->focus_window);
>   	}
>   }



More information about the wayland-devel mailing list