[FFmpeg-devel] [PATCH] avformat/ivfenc: Encode the number of frames

[James Almer]

On 10/1/2019 11:59 AM, Raphaël Zumer wrote:
> Signed-off-by: Raphaël Zumer <rzumer at tebako.net>
> ---
>  libavformat/ivfenc.c  | 3 ++-
>  libavformat/version.h | 2 +-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c
> index adf72117e9..54327f5025 100644
> --- a/libavformat/ivfenc.c
> +++ b/libavformat/ivfenc.c
> @@ -53,7 +53,8 @@ static int ivf_write_header(AVFormatContext *s)
>      avio_wl16(pb, par->height);
>      avio_wl32(pb, s->streams[0]->time_base.den);
>      avio_wl32(pb, s->streams[0]->time_base.num);
> -    avio_wl64(pb, 0xFFFFFFFFFFFFFFFFULL);
> +    avio_wl32(pb, s->streams[0]->nb_frames);
> +    avio_wl32(pb, 0xFFFFFFFFUL);

This field is overwritten at the end of the muxing process. The
UINT64_MAX value here is simply a placeholder.

This patch is wrong.