[FFmpeg-devel] [PATCH 01/13] avformat/mxfenc: Correct KAG alignment of preface

[Tomas Härdin]

mån 2018-05-07 klockan 12:38 +0200 skrev Michael Niedermayer:
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/mxfenc.c            |  1 +
>  tests/ref/fate/copy-trac4914    |  4 ++--
>  tests/ref/fate/mxf-reel_name    |  2 +-
>  tests/ref/fate/time_base        |  2 +-
>  tests/ref/lavf/mxf              | 12 ++++++------
>  tests/ref/lavf/mxf_d10          |  4 ++--
>  tests/ref/lavf/mxf_dv25         |  4 ++--
>  tests/ref/lavf/mxf_dvcpro50     |  4 ++--
>  tests/ref/lavf/mxf_opatom       |  2 +-
>  tests/ref/lavf/mxf_opatom_audio |  4 ++--
>  10 files changed, 20 insertions(+), 19 deletions(-)
> 
> diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> index 3bb70326fe..c0db10b3c2 100644
> --- a/libavformat/mxfenc.c
> +++ b/libavformat/mxfenc.c
> @@ -1757,6 +1757,7 @@ static int mxf_write_partition(AVFormatContext
> *s, int bodysid,
>          mxf_write_klv_fill(s);
>          start = avio_tell(s->pb);
>          mxf_write_primer_pack(s);
> +        mxf_write_klv_fill(s);
>          mxf_write_header_metadata_sets(s);
>          pos = avio_tell(s->pb);
>          header_byte_count = pos - start + klv_fill_size(pos);

Feels like such an elementary error. Probably OK, but it's been a while
since I read the specs

/Tomas