On Mon, 2 Aug 2021, Tomas Härdin wrote:

sön 2021-08-01 klockan 03:55 +0200 skrev Marton Balint:
The code expects every kind of metadata set to start with the generic metadata
set attributes.

Signed-off-by: Marton Balint <c...@passwd.hu>
---
 libavformat/mxfdec.c | 66 ++++++++++++++------------------------------
 1 file changed, 21 insertions(+), 45 deletions(-)

diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
index fd22680adb..34cbd2cd77 100644
--- a/libavformat/mxfdec.c
+++ b/libavformat/mxfdec.c
@@ -106,17 +106,19 @@ typedef struct MXFPartition {
     KLVPacket first_essence_klv;
 } MXFPartition;

-typedef struct MXFCryptoContext {
+typedef struct MXFMetadataSet {
     UID uid;
     MXFPartition *partition;
     enum MXFMetadataSetType type;
+} MXFMetadataSet;
+
+typedef struct MXFCryptoContext {
+    MXFMetadataSet meta;
     UID source_container_ul;
 } MXFCryptoContext;


This is something I've had in mind as well, but never got around to. I
approve! This also makes the code more strict-aliasing correct

Thanks, applied.

Regards,
Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to