diff mbox series

[FFmpeg-devel] avformat/mov: reword log message about dref

Message ID tencent_FC3469816234BF3F2D56139D9EE0F85B8009@qq.com
State New
Headers show
Series [FFmpeg-devel] avformat/mov: reword log message about dref | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

Zhao Zhili Sept. 2, 2021, 8:26 a.m. UTC 
Most of the times it's just 'url ', so it's not unknown.
---
By the way, there is a patch to add support for dref.url
https://patchwork.ffmpeg.org/project/ffmpeg/patch/1596807454-17939-2-git-send-email-mypopydev@gmail.com/
I agree there are safety issues.

 libavformat/mov.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/libavformat/mov.c b/libavformat/mov.c
index c5583e07c7..c4024fc402 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -708,8 +708,8 @@  static int mov_read_dref(MOVContext *c, AVIOContext *pb, MOVAtom atom)
                     avio_skip(pb, len);
             }
         } else {
-            av_log(c->fc, AV_LOG_DEBUG, "Unknown dref type 0x%08"PRIx32" size %"PRIu32"\n",
-                   dref->type, size);
+            av_log(c->fc, AV_LOG_DEBUG, "Ignore dref type %s size %"PRIu32"\n",
+                   av_fourcc2str(dref->type), size);
             entries--;
             i--;
         }