[FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the "number of frames" in IVF header

Anton Khirnov anton at khirnov.net
Wed Jun 28 18:24:59 EEST 2023


Quoting Dai, Jianhui J (2023-06-05 02:53:35)
> diff --git a/libavformat/ivfdec.c b/libavformat/ivfdec.c
> index 511f2387ed..01012db948 100644
> --- a/libavformat/ivfdec.c
> +++ b/libavformat/ivfdec.c
> @@ -53,6 +53,7 @@ static int read_header(AVFormatContext *s)
>      st->codecpar->height     = avio_rl16(s->pb);
>      time_base.den         = avio_rl32(s->pb);
>      time_base.num         = avio_rl32(s->pb);
> +    // Infer duration from "number of frames".
>      st->duration          = avio_rl32(s->pb);

This should be setting st->nb_frames then rather than duration.
And the muxer should be using that field as well instead of its custom
version.


-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list