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

[Carl Eugen Hoyos]

Am Di., 1. Okt. 2019 um 16:59 Uhr schrieb Raphaël Zumer <rzumer at tebako.net>:
>
> 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);

How did you test this patch?

It is wrong to write a time_base and the number of frames
like this, the output make no sense.

Carl Eugen