[PATCH weston v2] xwm: do not crash when activating a NULL surface

Kristian Høgsberg hoegsberg at gmail.com
Tue Aug 13 10:04:37 PDT 2013


On Tue, Aug 13, 2013 at 11:42:02AM +0200, Giulio Camuffo wrote:
> ---
>  src/xwayland/window-manager.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

Committed.

Kristian

> diff --git a/src/xwayland/window-manager.c b/src/xwayland/window-manager.c
> index d565888..4c9d477 100644
> --- a/src/xwayland/window-manager.c
> +++ b/src/xwayland/window-manager.c
> @@ -610,11 +610,15 @@ static void
>  weston_wm_window_activate(struct wl_listener *listener, void *data)
>  {
>  	struct weston_surface *surface = data;
> -	struct weston_wm_window *window = get_wm_window(surface);
> +	struct weston_wm_window *window = NULL;
>  	struct weston_wm *wm =
>  		container_of(listener, struct weston_wm, activate_listener);
>  	xcb_client_message_event_t client_message;
>  
> +	if (surface) {
> +		window = get_wm_window(surface);
> +	}
> +
>  	if (window) {
>  		client_message.response_type = XCB_CLIENT_MESSAGE;
>  		client_message.format = 32;
> -- 
> 1.8.3.4
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list