[vlc-devel] [PATCH] avahi: remove useless avahi_threaded_poll_stop() from error path

Jean-Baptiste Kempf jb at videolan.org
Wed Mar 9 11:49:12 CET 2016


On 09 Mar, Petri Hintukainen wrote :
> On ti, 2016-03-08 at 16:26 +0100, Jean-Baptiste Kempf wrote:
> > On 08 Mar, Petri Hintukainen wrote :
> > > Error path is never triggered after avahi_threaded_poll_start().
> > 
> > Right.
> > 
> > > Even if it was, thread should probably be stopped before freeing
> > > avahi client ... ?
> > 
> > Not so sure about that, tbh.
> 
> I'm not sure either. But it kind of makes sense to stop the thread
> before freeing data it may access. If avahi_client_free() is called
> while callback is running, callback may access already freed memory.

Yes. could make sense.

> This is how it is in the example ( 
> http://avahi.org/wiki/RunningAvahiClientAsThread ):

Then yes.

With my kindest regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device


More information about the vlc-devel mailing list