From 7de19a3264f530d14b509eff0c4a8d6e62c0f984 Mon Sep 17 00:00:00 2001 From: Stefano Sabatini Date: Mon, 22 Nov 2010 22:03:30 +0000 Subject: Implement robust parsing in aspect filters. Originally committed as revision 25802 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavfilter/vf_aspect.c | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) (limited to 'libavfilter/vf_aspect.c') diff --git a/libavfilter/vf_aspect.c b/libavfilter/vf_aspect.c index bb7690b7d2..97740f103d 100644 --- a/libavfilter/vf_aspect.c +++ b/libavfilter/vf_aspect.c @@ -34,22 +34,24 @@ static av_cold int init(AVFilterContext *ctx, const char *args, void *opaque) AspectContext *aspect = ctx->priv; double ratio; int64_t gcd; + char c = 0; if (args) { - if (sscanf(args, "%d:%d", &aspect->aspect.num, &aspect->aspect.den) < 2) { - if (sscanf(args, "%lf", &ratio) < 1) { - av_log(ctx, AV_LOG_ERROR, - "Invalid string '%s' for aspect ratio.\n", args); - return AVERROR(EINVAL); - } - aspect->aspect = av_d2q(ratio, 100); - } else { + if (sscanf(args, "%d:%d%c", &aspect->aspect.num, &aspect->aspect.den, &c) != 2) + if (sscanf(args, "%lf%c", &ratio, &c) == 1) + aspect->aspect = av_d2q(ratio, 100); + + if (c || aspect->aspect.num <= 0 || aspect->aspect.den <= 0) { + av_log(ctx, AV_LOG_ERROR, + "Invalid string '%s' for aspect ratio.\n", args); + return AVERROR(EINVAL); + } + gcd = av_gcd(FFABS(aspect->aspect.num), FFABS(aspect->aspect.den)); if (gcd) { aspect->aspect.num /= gcd; aspect->aspect.den /= gcd; } - } } if (aspect->aspect.den == 0) -- cgit v1.2.3