diff options
author | Tomas Härdin <tomas.hardin@codemill.se> | 2011-11-28 13:51:00 +0400 |
---|---|---|
committer | Janne Grunau <janne-libav@jannau.net> | 2012-01-22 17:40:52 +0400 |
commit | 0999151962f60abee1989d863dba6c09573b7a6b (patch) | |
tree | bfd4e046403749407e8df06f6517aa10c48aba06 /libavformat | |
parent | 0ac5e6a8c4d8223323c1a8c2801d41d1011c1995 (diff) |
mxfdec: Make sure DataDefinition is consistent between material track and source track.
This fixes 0001GL.MXF.V1.mxf_opatom.mxf and 0001GL00.MXF.A1.mxf_opatom.mxf
getting two streams each due to both using the same SourcePackageID.
Signed-off-by: Janne Grunau <janne-libav@jannau.net>
Diffstat (limited to 'libavformat')
-rw-r--r-- | libavformat/mxfdec.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 6f4824a44e..7c45b01c49 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -1153,6 +1153,18 @@ static int mxf_parse_structural_metadata(MXFContext *mxf) if (!source_track) continue; + if (!(source_track->sequence = mxf_resolve_strong_ref(mxf, &source_track->sequence_ref, Sequence))) { + av_log(mxf->fc, AV_LOG_ERROR, "could not resolve source track sequence strong ref\n"); + return AVERROR_INVALIDDATA; + } + + /* 0001GL00.MXF.A1.mxf_opatom.mxf has the same SourcePackageID as 0001GL.MXF.V1.mxf_opatom.mxf + * This would result in both files appearing to have two streams. Work around this by sanity checking DataDefinition */ + if (memcmp(material_track->sequence->data_definition_ul, source_track->sequence->data_definition_ul, 16)) { + av_log(mxf->fc, AV_LOG_ERROR, "material track %d: DataDefinition mismatch\n", material_track->track_id); + continue; + } + st = avformat_new_stream(mxf->fc, NULL); if (!st) { av_log(mxf->fc, AV_LOG_ERROR, "could not allocate stream\n"); @@ -1166,11 +1178,6 @@ static int mxf_parse_structural_metadata(MXFContext *mxf) st->start_time = component->start_position; avpriv_set_pts_info(st, 64, material_track->edit_rate.num, material_track->edit_rate.den); - if (!(source_track->sequence = mxf_resolve_strong_ref(mxf, &source_track->sequence_ref, Sequence))) { - av_log(mxf->fc, AV_LOG_ERROR, "could not resolve source track sequence strong ref\n"); - return AVERROR_INVALIDDATA; - } - PRINT_KEY(mxf->fc, "data definition ul", source_track->sequence->data_definition_ul); codec_ul = mxf_get_codec_ul(ff_mxf_data_definition_uls, &source_track->sequence->data_definition_ul); st->codec->codec_type = codec_ul->id; |