diff mbox series

[FFmpeg-devel] avformat/mxfdec: do not log warning of multiple ANC packets if count is 0

Message ID 20220804230338.13682-1-gcs584@yahoo.com
State New
Headers show
Series [FFmpeg-devel] avformat/mxfdec: do not log warning of multiple ANC packets if count is 0 | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Gavin Smith Aug. 4, 2022, 11:03 p.m. UTC
Some NLVEs may insert a KLV packet for EIA-608 data even though
the number of encapsulated ANC packets is zero.
---
 libavformat/mxfdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tomas Härdin Aug. 6, 2022, 8:44 p.m. UTC | #1
fre 2022-08-05 klockan 00:03 +0100 skrev Gavin Smith:
> Some NLVEs may insert a KLV packet for EIA-608 data even though
> the number of encapsulated ANC packets is zero.
> ---
>  libavformat/mxfdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Looks OK. Also I'm looking at maybe adding S436m support to mxfenc, but
it's going to require a bit of plumbing in ffmpeg.c I think.

/Tomas
diff mbox series

Patch

diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
index 400941c348..f3d2cf4efa 100644
--- a/libavformat/mxfdec.c
+++ b/libavformat/mxfdec.c
@@ -509,7 +509,7 @@  static int mxf_get_eia608_packet(AVFormatContext *s, AVStream *st, AVPacket *pkt
     int did, sdid, data_length;
     int i, ret;
 
-    if (count != 1)
+    if (count > 1)
         av_log(s, AV_LOG_WARNING, "unsupported multiple ANC packets (%d) per KLV packet\n", count);
 
     for (i = 0; i < count; i++) {