[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