From f589dac2b0298deeb92f4464645d702292425675 Mon Sep 17 00:00:00 2001 From: Daniel Stone Date: Thu, 16 Feb 2017 19:59:51 +0000 Subject: [PATCH] screen-share: Use wl_list_for_each_safe on destroy Destroying the shared seat removes the link from so->seat_list. Signed-off-by: Daniel Stone Reviewed-by: Pekka Paalanen Acked-by: Bryce Harrington --- 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.7.4