diff options
| author | Ganesh Ajjanagadde <gajjanagadde@gmail.com> | 2015-10-09 11:25:16 -0400 |
|---|---|---|
| committer | Ganesh Ajjanagadde <gajjanagadde@gmail.com> | 2015-10-13 19:40:09 -0400 |
| commit | 4c8ca76965b1c29758246624940cbc529e7141f0 (patch) | |
| tree | 7db7076d13519c7cd9b86e88365aaad504092584 | |
| parent | 74a87ae21075d694ada0d19705d9ec40b39d6037 (diff) | |
ffserver_config: check for INT_MIN before doing FFABS
FFABS(INT_MIN) is not safe. Alternative of using FFNABS is not as
readable.
Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
| -rw-r--r-- | ffserver_config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ffserver_config.c b/ffserver_config.c index 691ff7b49e..9fc1f0030c 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -460,7 +460,7 @@ static int ffserver_set_int_param(int *dest, const char *value, int factor, if (tmp < min || tmp > max) goto error; if (factor) { - if (FFABS(tmp) > INT_MAX / FFABS(factor)) + if (tmp == INT_MIN || FFABS(tmp) > INT_MAX / FFABS(factor)) goto error; tmp *= factor; } |