On Thu, Feb 16, 2017 at 07:59:51PM +0000, Daniel Stone wrote:
> Destroying the shared seat removes the link from so->seat_list.
> 
> Signed-off-by: Daniel Stone <dani...@collabora.com>

Acked-by: Bryce Harrington <br...@osg.samsung.com>

> ---
>  compositor/screen-share.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/compositor/screen-share.c b/compositor/screen-share.c
> index 069da1d..a6f82b1 100644
> --- a/compositor/screen-share.c
> +++ b/compositor/screen-share.c
> @@ -884,7 +884,7 @@ shared_output_create(struct weston_output *output, int 
> parent_fd)
>  {
>       struct shared_output *so;
>       struct wl_event_loop *loop;
> -     struct ss_seat *seat;
> +     struct ss_seat *seat, *tmp;
>       int epoll_fd;
>  
>       so = zalloc(sizeof *so);
> @@ -972,7 +972,7 @@ shared_output_create(struct weston_output *output, int 
> parent_fd)
>       return so;
>  
>  err_display:
> -     wl_list_for_each(seat, &so->seat_list, link)
> +     wl_list_for_each_safe(seat, tmp, &so->seat_list, link)
>               ss_seat_destroy(seat);
>       wl_display_disconnect(so->parent.display);
>  err_alloc:
> -- 
> 2.9.3
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to