diff options
| author | Tomas Härdin <tomas.hardin@codemill.se> | 2012-01-11 11:29:57 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2012-01-17 20:55:19 +0100 |
| commit | 1de5154dfb9a1987f0cfb597ca71a41425015417 (patch) | |
| tree | ca5b63ea6f6d6be09f6a4da6aad4a0448bd6e2e6 /libavformat/mxfdec.c | |
| parent | aff9cb8377aeeadeeb01090be3bb642c220d7a36 (diff) | |
| download | ffmpeg-1de5154dfb9a1987f0cfb597ca71a41425015417.tar.gz | |
mxfdec: Consider OPAtom files that don't have exactly one EC to be OP1a
This fixes demuxing of 2011_DCPTEST_24FPS.V.mxf.
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavformat/mxfdec.c')
| -rw-r--r-- | libavformat/mxfdec.c | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 4c2949d7fc..e5574e2d18 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -59,7 +59,7 @@ typedef enum { } MXFPartitionType; typedef enum { - OP1a, + OP1a = 1, OP1b, OP1c, OP2a, @@ -410,6 +410,7 @@ static int mxf_read_partition_pack(void *arg, AVIOContext *pb, int tag, int size MXFPartition *partition; UID op; uint64_t footer_partition; + uint32_t nb_essence_containers; if (mxf->partitions_count+1 >= UINT_MAX / sizeof(*mxf->partitions)) return AVERROR(ENOMEM); @@ -463,6 +464,7 @@ static int mxf_read_partition_pack(void *arg, AVIOContext *pb, int tag, int size avio_skip(pb, 8); partition->body_sid = avio_rb32(pb); avio_read(pb, op, sizeof(UID)); + nb_essence_containers = avio_rb32(pb); /* some files don'thave FooterPartition set in every partition */ if (footer_partition) { @@ -496,9 +498,19 @@ static int mxf_read_partition_pack(void *arg, AVIOContext *pb, int tag, int size else if (op[12] == 3 && op[13] == 1) mxf->op = OP3a; else if (op[12] == 3 && op[13] == 2) mxf->op = OP3b; else if (op[12] == 3 && op[13] == 3) mxf->op = OP3c; - else if (op[12] == 0x10) mxf->op = OPAtom; else if (op[12] == 64&& op[13] == 1) mxf->op = OPSONYOpt; - else { + else if (op[12] == 0x10) { + /* SMPTE 390m: "There shall be exactly one essence container" + * 2011_DCPTEST_24FPS.V.mxf violates this and is frame wrapped, hence why we assume OP1a */ + if (nb_essence_containers != 1) { + /* only nag once */ + if (!mxf->op) + av_log(mxf->fc, AV_LOG_WARNING, "\"OPAtom\" with %u ECs - assuming OP1a\n", nb_essence_containers); + + mxf->op = OP1a; + } else + mxf->op = OPAtom; + } else { av_log(mxf->fc, AV_LOG_ERROR, "unknown operational pattern: %02xh %02xh - guessing OP1a\n", op[12], op[13]); mxf->op = OP1a; } |