On Thu, 16 Feb 2017 19:59:51 +0000 Daniel Stone <dani...@collabora.com> wrote:
> Destroying the shared seat removes the link from so->seat_list.
>
> Signed-off-by: Daniel Stone <dani...@collabora.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:
Both patches:
Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk>
And also look fine to land for the release, IMO.
Thanks,
pq
pgpwyy_CHzMpB.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel
