[FFmpeg-devel] [PATCH 1/2] avformat/concatdec: set safe mode to enabled instead of auto

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Wed Jan 20 00:03:59 CET 2016


On 19.01.2016 22:13, Michael Niedermayer wrote:
> From: Michael Niedermayer <michael at niedermayer.cc>
> 
> This is safer, as a selected demuxer could still mean that it was auto-detected
> by a user application
> 
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/concatdec.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c
> index d226e15..de7b89a 100644
> --- a/libavformat/concatdec.c
> +++ b/libavformat/concatdec.c
> @@ -710,7 +710,7 @@ static int concat_seek(AVFormatContext *avf, int stream,
>  
>  static const AVOption options[] = {
>      { "safe", "enable safe mode",
> -      OFFSET(safe), AV_OPT_TYPE_BOOL, {.i64 = -1}, -1, 1, DEC },
> +      OFFSET(safe), AV_OPT_TYPE_BOOL, {.i64 = 1}, -1, 1, DEC },
>      { "auto_convert", "automatically convert bitstream format",
>        OFFSET(auto_convert), AV_OPT_TYPE_BOOL, {.i64 = 1}, 0, 1, DEC },
>      { "segment_time_metadata", "output file segment start time and duration as packet metadata",
> 

The change seems OK, but the documentation in doc/demuxers.texi should be updated
accordingly.

Best regards,
Andreas


More information about the ffmpeg-devel mailing list