[FFmpeg-devel] [PATCH] avutil/timecode: fix sscanf format string with garbage at the end

lance.lmwang at gmail.com lance.lmwang at gmail.com
Sat Jun 20 18:17:23 EEST 2020


From: Limin Wang <lance.lmwang at gmail.com>

Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
---
 libavutil/timecode.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavutil/timecode.c b/libavutil/timecode.c
index 60077ba..1facc41 100644
--- a/libavutil/timecode.c
+++ b/libavutil/timecode.c
@@ -196,7 +196,7 @@ int av_timecode_init_from_string(AVTimecode *tc, AVRational rate, const char *st
     char c;
     int hh, mm, ss, ff, ret;
 
-    if (sscanf(str, "%d:%d:%d%c%d", &hh, &mm, &ss, &c, &ff) != 5) {
+    if (sscanf(str, "%02u:%02u:%02u%c%02u", &hh, &mm, &ss, &c, &ff) != 5) {
         av_log(log_ctx, AV_LOG_ERROR, "Unable to parse timecode, "
                                       "syntax: hh:mm:ss[:;.]ff\n");
         return AVERROR_INVALIDDATA;
-- 
1.8.3.1



More information about the ffmpeg-devel mailing list