diff mbox

[FFmpeg-devel] [PATCH}lavf/mxfdec: Search all components of material track for source package

Message ID 201611050150.37926.cehoyos@ag.or.at
State Accepted
Commit 4590d073ccdc7c3ce0384d5b1fba56b3f6673535
Headers show

Commit Message

Carl Eugen Hoyos Nov. 5, 2016, 12:50 a.m. UTC
Hi!

Attached patch fixes ticket #5925 here.

Please review, Carl Eugen
From c928270086e6021fba7d6d3201be7ea569e37253 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <cehoyos@ag.or.at>
Date: Sat, 5 Nov 2016 01:48:10 +0100
Subject: [PATCH] lavf/mxfdec: Search all components of material track for
 source package.

Fixes ticket #5925.
---
 libavformat/mxfdec.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Carl Eugen Hoyos Sept. 20, 2017, 11:35 a.m. UTC | #1
2016-11-05 1:50 GMT+01:00 Carl Eugen Hoyos <cehoyos@ag.or.at>:
> Hi!
>
> Attached patch fixes ticket #5925 here.

I'll push this if there are no comments.

Carl Eugen
Carl Eugen Hoyos Oct. 3, 2017, 9:30 p.m. UTC | #2
2016-11-05 1:50 GMT+01:00 Carl Eugen Hoyos <cehoyos@ag.or.at>:
> Hi!
>
> Attached patch fixes ticket #5925 here.

Ok'ed by Marton and pushed.

Carl Eugen
diff mbox

Patch

diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
index d2166ee..c687ce5 100644
--- a/libavformat/mxfdec.c
+++ b/libavformat/mxfdec.c
@@ -1974,7 +1974,7 @@  static int mxf_parse_structural_metadata(MXFContext *mxf)
             source_package = mxf_resolve_source_package(mxf, component->source_package_uid);
             if (!source_package) {
                 av_log(mxf->fc, AV_LOG_TRACE, "material track %d: no corresponding source package found\n", material_track->track_id);
-                break;
+                continue;
             }
             for (k = 0; k < source_package->tracks_count; k++) {
                 if (!(temp_track = mxf_resolve_strong_ref(mxf, &source_package->tracks_refs[k], Track))) {