[PATCH weston 2/2] screen-share: Use wl_list_for_each_safe on destroy

[Bryce Harrington]

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 <daniels at collabora.com>

Acked-by: Bryce Harrington <bryce at 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 at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel