<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 8, 2020 at 8:40 PM Marvin Scholz <<a href="mailto:epirat07@gmail.com">epirat07@gmail.com</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"><br>
<br>
On 8 Apr 2020, at 20:33, Mehdi Sabwat wrote:<br>
<br>
> From: Mehdi Sabwat <<a href="mailto:mehdisabwat@gmail.com" target="_blank">mehdisabwat@gmail.com</a>><br>
><br>
> Since Jan 21, 2018 (ea3672b7d67c432724bdbc8de0221f869b6a04c6)<br>
> filename field was deprecated in favor of `url` which now has<br>
> no length restrictions.<br>
><br>
> ```<br>
> 2018-01-28 - ea3672b7d6 - lavf 58.7.100 - avformat.h<br>
>   Deprecate AVFormatContext filename field which had limited length, <br>
> use the<br>
>   new dynamically allocated url field instead.<br>
> ```<br>
> ---<br>
>  modules/demux/avformat/mux.c | 6 ++++++<br>
>  1 file changed, 6 insertions(+)<br>
><br>
> diff --git a/modules/demux/avformat/mux.c <br>
> b/modules/demux/avformat/mux.c<br>
> index 9179daf592..35ffb4c517 100644<br>
> --- a/modules/demux/avformat/mux.c<br>
> +++ b/modules/demux/avformat/mux.c<br>
> @@ -90,8 +90,10 @@ int avformat_OpenMux( vlc_object_t *p_this )<br>
>      sout_mux_t *p_mux = (sout_mux_t*)p_this;<br>
>      bool dummy = !strcmp( p_mux->p_access->psz_access, "dummy");<br>
><br>
> +#if (LIBAVFORMAT_VERSION_INT < AV_VERSION_INT(58, 7, 100))<br>
>      if( dummy && strlen(p_mux->p_access->psz_path)<br>
>                                >= sizeof (((AVFormatContext <br>
> *)NULL)->filename) )<br>
> +#endif<br>
>          return VLC_EGENERIC;<br>
><br>
<br>
This will unconditionally return VLC_EGENERIC with the older versions,<br>
probably not what you intended to do.<br></blockquote><div><br>Thank you for the review,  yes completely missed that too.<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
>      msg_Dbg( p_mux, "using %s %s", AVPROVIDER(LIBAVFORMAT), <br>
> LIBAVFORMAT_IDENT );<br>
> @@ -127,7 +129,11 @@ int avformat_OpenMux( vlc_object_t *p_this )<br>
>      p_sys->oc->oformat = file_oformat;<br>
>      /* If we use dummy access, let avformat write output */<br>
>      if( dummy )<br>
> +#if (LIBAVFORMAT_VERSION_INT >= AV_VERSION_INT(58, 7, 100))<br>
> +        p_sys->oc->url = av_strdup(p_mux->p_access->psz_path);<br>
> +#else<br>
>          strcpy( p_sys->oc->filename, p_mux->p_access->psz_path );<br>
> +#endif<br>
<br>
This does not seem to check libav properly, just ffmpeg?<br>
<br></blockquote><div><br></div><div>true, libav still has the filename field. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
><br>>      /* Create I/O wrapper */<br>>      p_sys->io_buffer_size = 10 * 1024 * 1024;  /* FIXME */> -- <br>
> 2.25.0<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>
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><br></div><div>Sending the corrected version in reply. </div></div></div>