avformat/mxfdec: do not use AnyType when resolving Descriptors and MultipleDescriptors

By using AnyType for resolving a strong reference we searched among all types,
not just the ones which can be the target of the reference, which in some cases
caused to find the wrong type, if the metadata set UUIDs were not unique.

UUIDs do not have to be unique if their type sets them apart, SMPTE 377M says:

> StrongRef: 'One to One’ relationship between sets and implemented in MXF
> with UUIDs. Strong References are typed which means that the definition
> identifies the kind of set which is the target of the reference.

Fixes ticket #10865.

Signed-off-by: Marton Balint <cus@passwd.hu>
release/7.0
Marton Balint 9 months ago
parent 41672f5586
commit 68f2b32ef2
  1. 19
      libavformat/mxfdec.c

@ -2258,16 +2258,14 @@ static MXFPackage* mxf_resolve_source_package(MXFContext *mxf, UID package_ul, U
static MXFDescriptor* mxf_resolve_descriptor(MXFContext *mxf, UID *strong_ref, int track_id)
{
MXFDescriptor *descriptor, *file_descriptor = NULL;
int i;
descriptor = mxf_resolve_strong_ref(mxf, strong_ref, AnyType);
if (!descriptor)
return NULL;
MXFDescriptor *descriptor = mxf_resolve_strong_ref(mxf, strong_ref, Descriptor);
if (descriptor)
return descriptor;
if (descriptor->meta.type == MultipleDescriptor) {
for (i = 0; i < descriptor->file_descriptors_count; i++) {
file_descriptor = mxf_resolve_strong_ref(mxf, &descriptor->file_descriptors_refs[i], Descriptor);
descriptor = mxf_resolve_strong_ref(mxf, strong_ref, MultipleDescriptor);
if (descriptor) {
for (int i = 0; i < descriptor->file_descriptors_count; i++) {
MXFDescriptor *file_descriptor = mxf_resolve_strong_ref(mxf, &descriptor->file_descriptors_refs[i], Descriptor);
if (!file_descriptor) {
av_log(mxf->fc, AV_LOG_ERROR, "could not resolve file descriptor strong ref\n");
@ -2277,8 +2275,7 @@ static MXFDescriptor* mxf_resolve_descriptor(MXFContext *mxf, UID *strong_ref, i
return file_descriptor;
}
}
} else if (descriptor->meta.type == Descriptor)
return descriptor;
}
return NULL;
}

Loading…
Cancel
Save