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