Commit 7de19a32 authored by Stefano Sabatini's avatar Stefano Sabatini

Implement robust parsing in aspect filters.

Originally committed as revision 25802 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent eee0ef5e
...@@ -34,23 +34,25 @@ static av_cold int init(AVFilterContext *ctx, const char *args, void *opaque) ...@@ -34,23 +34,25 @@ static av_cold int init(AVFilterContext *ctx, const char *args, void *opaque)
AspectContext *aspect = ctx->priv; AspectContext *aspect = ctx->priv;
double ratio; double ratio;
int64_t gcd; int64_t gcd;
char c = 0;
if (args) { if (args) {
if (sscanf(args, "%d:%d", &aspect->aspect.num, &aspect->aspect.den) < 2) { if (sscanf(args, "%d:%d%c", &aspect->aspect.num, &aspect->aspect.den, &c) != 2)
if (sscanf(args, "%lf", &ratio) < 1) { 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, av_log(ctx, AV_LOG_ERROR,
"Invalid string '%s' for aspect ratio.\n", args); "Invalid string '%s' for aspect ratio.\n", args);
return AVERROR(EINVAL); return AVERROR(EINVAL);
} }
aspect->aspect = av_d2q(ratio, 100);
} else {
gcd = av_gcd(FFABS(aspect->aspect.num), FFABS(aspect->aspect.den)); gcd = av_gcd(FFABS(aspect->aspect.num), FFABS(aspect->aspect.den));
if (gcd) { if (gcd) {
aspect->aspect.num /= gcd; aspect->aspect.num /= gcd;
aspect->aspect.den /= gcd; aspect->aspect.den /= gcd;
} }
} }
}
if (aspect->aspect.den == 0) if (aspect->aspect.den == 0)
aspect->aspect = (AVRational) {0, 1}; aspect->aspect = (AVRational) {0, 1};
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment