openharmony/third_party_ffmpeg
1
0
Fork 0
mirror of https://gitee.com/openharmony/third_party_ffmpeg synced 2024-11-30 06:30:59 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

mxfdec: Consider OPAtom files that don't have exactly one EC to be OP1a

Browse Source 
This fixes demuxing of 2011_DCPTEST_24FPS.V.mxf.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
This commit is contained in:
Tomas Härdin 2012-01-11 11:29:57 +01:00 committed by Michael Niedermayer
parent aff9cb8377
commit 1de5154dfb
1 changed files with 15 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
libavformat/mxfdec.c
View File

@ -59,7 +59,7 @@ typedef enum {
} MXFPartitionType; } MXFPartitionType;
typedef enum { typedef enum {
OP1a, OP1a = 1,
OP1b, OP1b,
OP1c, OP1c,
OP2a, OP2a,
@ -410,6 +410,7 @@ static int mxf_read_partition_pack(void *arg, AVIOContext *pb, int tag, int size
MXFPartition *partition; MXFPartition *partition;
UID op; UID op;
uint64_t footer_partition; uint64_t footer_partition;
uint32_t nb_essence_containers;
if (mxf->partitions_count+1 >= UINT_MAX / sizeof(*mxf->partitions)) if (mxf->partitions_count+1 >= UINT_MAX / sizeof(*mxf->partitions))
return AVERROR(ENOMEM); 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); avio_skip(pb, 8);
partition->body_sid = avio_rb32(pb); partition->body_sid = avio_rb32(pb);
avio_read(pb, op, sizeof(UID)); avio_read(pb, op, sizeof(UID));
nb_essence_containers = avio_rb32(pb);
/* some files don'thave FooterPartition set in every partition */ /* some files don'thave FooterPartition set in every partition */
if (footer_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] == 1) mxf->op = OP3a;
else if (op[12] == 3 && op[13] == 2) mxf->op = OP3b; 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] == 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 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]); av_log(mxf->fc, AV_LOG_ERROR, "unknown operational pattern: %02xh %02xh - guessing OP1a\n", op[12], op[13]);
mxf->op = OP1a; mxf->op = OP1a;
} }