From 29f09fe876845991d5e1924f975715e3b57295d8 Mon Sep 17 00:00:00 2001 From: Bryce Harrington Date: Fri, 10 Jul 2015 18:52:56 -0700 Subject: [PATCH] screen-share: Use the correct list pointer when freeing shm buffers The wl_list_for_each operation on the free_buffers list should use free_link not link, which is a different list. Signed-off-by: Bryce Harrington Reviewed-by: Derek Foreman --- src/screen-share.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/screen-share.c b/src/screen-share.c index 6b1b34ccf..10dc12831 100644 --- a/src/screen-share.c +++ b/src/screen-share.c @@ -414,7 +414,7 @@ shared_output_get_shm_buffer(struct shared_output *so) so->shm.height != height) { /* Destroy free buffers */ - wl_list_for_each_safe(sb, bnext, &so->shm.free_buffers, link) + wl_list_for_each_safe(sb, bnext, &so->shm.free_buffers, free_link) ss_shm_buffer_destroy(sb); /* Orphan in-use buffers so they get destroyed */ @@ -975,7 +975,7 @@ shared_output_destroy(struct shared_output *so) wl_list_for_each_safe(buffer, bnext, &so->shm.buffers, link) ss_shm_buffer_destroy(buffer); - wl_list_for_each_safe(buffer, bnext, &so->shm.free_buffers, link) + wl_list_for_each_safe(buffer, bnext, &so->shm.free_buffers, free_link) ss_shm_buffer_destroy(buffer); wl_display_disconnect(so->parent.display);