[vlc-devel] Using logger during asynchronous close
remi at remlab.net
Mon Mar 8 16:45:32 UTC 2021
Le lundi 8 mars 2021, 17:56:54 EET Alexandre Janniaux a écrit :
> After merging my last mediacodec patches, I encountered segmentation
> faults that unfortunately never occurred previously but were bound to
> Indeed, after those patches, the mediacodec decoder_t object can be
> closed while the mediacodec core is still valid as long as the video
> context is valid. If the mediacodec core is logging message, it will
> go through the closed and potentially destroyed decoder_t object.
> I'm looking for solution for this problem but I'm not sure we already
> encountered this issue so I'm not sure how to solve it.
> So far, I have those solutions:
> - adding back refcount on vlc_object_t, which would probably be
That wouldn't just be horrible. It wouldn't work. It only guaranteed that
object variables were still usable, which is not what you would need.
> - adding refcount to logger object. We basically know that the current
> logger is always available but it might change in the future so
> having a way to ensure it's validity in a better way than having a
> way to ensure the validity of the underlying object.
That wouldn't work either. You can't simply slap a reference count on
something that does not have one. You'd merely move the problem from
preserving the logger structure to preserving the underlying resources of the
logger backend, and break the assumptions on the life cycle of those
> - creating a dedicated object to log onto from libvlc, but then I lose
> the benefit of the logger inheritency infrastructure.
> - likewise, using the logger from the libvlc instance, but same problem.
> - using the logger from the object, but it could disappear too.
You can't have it both ways. You can only log to a backend whose life cycle
encompasses the source of the log messages.
Either you fix the design not to perform in(s)ane things, or you devise a weak
reference mechanism. The later means that log messages will sometime get lost,
which is far from good.
More information about the vlc-devel