[PATCH weston v2] shell: clear popup grab interface on remove_popup_grab and popup_grab_end

Kristian Høgsberg hoegsberg at gmail.com
Mon Apr 15 18:36:14 PDT 2013


On Mon, Apr 15, 2013 at 09:09:54PM +0200, Philipp Brüschweiler wrote:
> Fixes a segfault. Steps to reproduce:
> 
> * start weston with the x11 backend
> * open a terminal
> * click on the icon in the top left corner, choose close
> * close the x11 window containing weston
> ---
>  src/shell.c | 3 +++
>  1 file changed, 3 insertions(+)

Ok, I see.  This is to make sure the destroy_shell_seat() code doesn't
kick in if the popup grab is no longer active.  Thanks for tracking it
down, I've seen the crashes too and I'm happy to see it fixed.

I think we'll need to have a second look at that code path after 1.1
is out.  If the entire seat goes away, we do want to send out
popup_done, and we can reuse popup_grab_end() in destroy_shell_seat()
then.

Kristian

> diff --git a/src/shell.c b/src/shell.c
> index de5d6f6..9d0ae02 100644
> --- a/src/shell.c
> +++ b/src/shell.c
> @@ -1986,6 +1986,8 @@ popup_grab_end(struct wl_pointer *pointer)
>  	if (pointer->grab->interface == &popup_grab_interface) {
>  		wl_pointer_end_grab(grab->pointer);
>  		shseat->popup_grab.client = NULL;
> +		shseat->popup_grab.grab.interface = NULL;
> +		assert(!wl_list_empty(&shseat->popup_grab.surfaces_list));
>  		/* Send the popup_done event to all the popups open */
>  		wl_list_for_each(shsurf, &shseat->popup_grab.surfaces_list, popup.grab_link) {
>  			wl_shell_surface_send_popup_done(&shsurf->resource);
> @@ -2028,6 +2030,7 @@ remove_popup_grab(struct shell_surface *shsurf)
>  	wl_list_init(&shsurf->popup.grab_link);
>  	if (wl_list_empty(&shseat->popup_grab.surfaces_list)) {
>  		wl_pointer_end_grab(shseat->popup_grab.grab.pointer);
> +		shseat->popup_grab.grab.interface = NULL;
>  	}
>  }
>  
> -- 
> 1.8.2.1
> 
> _______________________________________________
> 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