[FFmpeg-devel] [PATCH] lavu/parseutils: add trailing characters check in av_parse_video_size()

Stefano Sabatini stefasab at gmail.com
Mon Oct 29 16:40:29 CET 2012


Return an error in case the video size specifications contains spurious
trailing chars, like in "320x240foobar".
---
 libavutil/parseutils.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c
index 0a7a622..a335978 100644
--- a/libavutil/parseutils.c
+++ b/libavutil/parseutils.c
@@ -143,6 +143,10 @@ int av_parse_video_size(int *width_ptr, int *height_ptr, const char *str)
         if (*p)
             p++;
         height = strtol(p, (void*)&p, 10);
+
+        /* trailing extraneous data detected, like in 123x345foobar */
+        if (*p)
+            return AVERROR(EINVAL);
     }
     if (width <= 0 || height <= 0)
         return AVERROR(EINVAL);
-- 
1.7.5.4



More information about the ffmpeg-devel mailing list