<div dir="ltr"><div>Hi Thomas,</div><div>That was done in my previous patch so please look into that too.</div><div>If I may suggest, the latest stable release does not clean the decoder in the call to DeleteSoutStreamID(), and I don't see why it currently does. Would it be better to just not clean the decoder in DeleteSoutStreamID()?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 20, 2019 at 7:52 PM Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u><div><div>I think you should set p_module and description to NULL for spu and video like it's done for audio.<br></div><div><br></div><div>On Wed, Mar 20, 2019, at 15:46, Awais Chishti wrote:<br></div><blockquote type="cite" id="gmail-m_6423419013656679381fastmail-quoted"><div dir="ltr"><div>Hi,<br></div><div>Can you please specify whether you're saying that the crash is impossible or that the workaround can not be implemented<br></div><div>I'm new to the code so it isn't explanatory from your reason<br></div></div><div><br></div><div class="gmail-m_6423419013656679381fastmail-quoted-gmail_quote"><div class="gmail-m_6423419013656679381fastmail-quoted-gmail_attr" dir="ltr">On Wed, Mar 20, 2019 at 7:32 PM Francois Cartegnie <<a href="mailto:fcartegnie@gmail.com" target="_blank">fcartegnie@gmail.com</a>> wrote:<br></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail-m_6423419013656679381fastmail-quoted-gmail_quote"><div>Le 20/03/2019 à 18:08, <a href="mailto:chishtiawais511@gmail.com" target="_blank">chishtiawais511@gmail.com</a> a écrit :<br></div><div> > From: Awais Chishti <<a href="mailto:chishtiawais511@gmail.com" target="_blank">chishtiawais511@gmail.com</a>><br></div><div> > <br></div><div> > 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.<br></div><div> > ---<br></div><div> <br></div><div> <br></div><div> AFAIR that's not possible as the decoder probing must<br></div><div> copy the resulting format while the decoder is not touching it.<br></div><div> <br></div><div> <br></div><div> -- <br></div><div> Francois Cartegnie<br></div><div> VideoLAN - VLC Developer<br></div><div> _______________________________________________<br></div><div> vlc-devel mailing list<br></div><div> To unsubscribe or modify your subscription options:<br></div><div> <a rel="noreferrer" href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div><a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote><div><br></div></div>_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>