[PATCH] screen-share: Use the correct list pointer when freeing shm buffers

[Bryce Harrington]

On Mon, Jul 13, 2015 at 11:02:07AM -0500, Derek Foreman wrote:
> looks correct to me.
> 
> Reviewed-by: Derek Foreman <derekf at osg.samsung.com>

Thanks, pushed:
To ssh://git.freedesktop.org/git/wayland/weston
   2f2a70c..29f09fe  master -> master
 
> On 10/07/15 08:52 PM, Bryce Harrington wrote:
> > 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 <bryce at osg.samsung.com>
> > ---
> >  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 6b1b34c..10dc128 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);
> >