lavfi/aspect: check for a negative code from av_parse_ratio()
authorStefano Sabatini <stefasab@gmail.com>
Wed, 21 Mar 2012 13:39:24 +0000 (14:39 +0100)
committerStefano Sabatini <stefasab@gmail.com>
Wed, 21 Mar 2012 14:14:28 +0000 (15:14 +0100)
Check on < 0 rather than on != 0, this is more correct as a positive
error code from av_parse_ratio() value doesn't mean an error.

libavfilter/vf_aspect.c

index 9e3b93b..8a56116 100644 (file)
@@ -37,7 +37,7 @@ static av_cold int init(AVFilterContext *ctx, const char *args, void *opaque)
     aspect->ratio = (AVRational) {0, 1};
 
     if (args) {
-        if (av_parse_ratio(&aspect->ratio, args, 100, 0, ctx) ||
+        if (av_parse_ratio(&aspect->ratio, args, 100, 0, ctx) < 0 ||
             aspect->ratio.num < 0 || aspect->ratio.den <= 0) {
             av_log(ctx, AV_LOG_ERROR,
                    "Invalid string '%s' for aspect ratio.\n", args);