[vlc-devel] [PATCH] Fix double free in stream deletion

Steve Lhomme robux4 at ycbcr.xyz
Thu Mar 21 09:20:34 CET 2019


I'd like Francois to clarify what he meant.

On 3/21/2019 8:32 AM, Awais Chishti wrote:
> So, what do we do to fix it?
>
> On Wed, Mar 20, 2019 at 7:58 PM Steve Lhomme <robux4 at ycbcr.xyz 
> <mailto:robux4 at ycbcr.xyz>> wrote:
>
>     On 3/20/2019 3:51 PM, Thomas Guillem wrote:
>     > I think you should set p_module and description to NULL for spu and
>     > video like it's done for audio.
>
>     That's what the previous patch was doing. But IMO it's not correct.
>     Although I don't understand what Francois is referring to.
>
>     It could be the decoder must be released before the rest of the
>     *_clean() function is done, but I don't see anything in there that
>     looks
>     dubious.
>
>     >
>     > On Wed, Mar 20, 2019, at 15:46, Awais Chishti wrote:
>     >> Hi,
>     >> Can you please specify whether you're saying that the crash is
>     >> impossible or that the workaround can not be implemented
>     >> I'm new to the code so it isn't explanatory from your reason
>     >>
>     >> On Wed, Mar 20, 2019 at 7:32 PM Francois Cartegnie
>     >> <fcartegnie at gmail.com <mailto:fcartegnie at gmail.com>
>     <mailto:fcartegnie at gmail.com <mailto:fcartegnie at gmail.com>>> wrote:
>     >>
>     >>     Le 20/03/2019 à 18:08, chishtiawais511 at gmail.com
>     <mailto:chishtiawais511 at gmail.com>
>     >>     <mailto:chishtiawais511 at gmail.com
>     <mailto:chishtiawais511 at gmail.com>> a écrit :
>     >>     > From: Awais Chishti <chishtiawais511 at gmail.com
>     <mailto:chishtiawais511 at gmail.com>
>     >>     <mailto:chishtiawais511 at gmail.com
>     <mailto:chishtiawais511 at gmail.com>>>
>     >>     >
>     >>     > Decoder cleaning is being done with module_unneed() once in
>     >>     transcode.c:697 and sometimes in the preceeding switch
>     statement
>     >>     causing VLC to crash. Either avoid cleaning the decoder in
>     >>     DeleteSoutStreamID() or in the transcode_*() functions. The
>     patch
>     >>     does the latter as suggested earlier.
>     >>     > ---
>     >>
>     >>
>     >>     AFAIR that's not possible as the decoder probing must
>     >>     copy the resulting format while the decoder is not touching it.
>     >>
>     >>
>     >>     --
>     >>     Francois Cartegnie
>     >>     VideoLAN - VLC Developer
>     >>     _______________________________________________
>     >>     vlc-devel mailing list
>     >>     To unsubscribe or modify your subscription options:
>     >> https://mailman.videolan.org/listinfo/vlc-devel
>     >>
>     >> _______________________________________________
>     >> vlc-devel mailing list
>     >> To unsubscribe or modify your subscription options:
>     >> https://mailman.videolan.org/listinfo/vlc-devel
>     >
>     >
>     > _______________________________________________
>     > vlc-devel mailing list
>     > To unsubscribe or modify your subscription options:
>     > https://mailman.videolan.org/listinfo/vlc-devel
>
>     _______________________________________________
>     vlc-devel mailing list
>     To unsubscribe or modify your subscription options:
>     https://mailman.videolan.org/listinfo/vlc-devel
>
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel



More information about the vlc-devel mailing list