[FFmpeg-devel] [PATCH] mxfdec: Always compute essence container lengths

[Alexander Strasser]

Hi

Tomas Härdin wrote:
[...]
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 10512ef..929a331 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
[...]
> @@ -1652,6 +1648,9 @@ static int mxf_read_header(AVFormatContext *s, AVFormatParameters *ap)
>                  if (mxf->op == OPAtom) {
>                      /* point essence_offset to the actual data
>                      * OPAtom has all the essence in one big KLV
> +                    * NOTE: Some "OPAtom" files like 2011_DCPTEST_24FPS.V.mxf are frame wrapped.
> +                    * For these files essence_length will be BS, fixed by mxf_compute_essence_containers().
> +                    * We still need to essence_length set for OPAtom files that lack a footer though.
                                          ^^^^^^^^^^^^^^^^^^
  Looks like the marked words got out of order.

[...]

  Alexander