aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2015-05-15 16:58:51 +0200
committerMichael Niedermayer <michaelni@gmx.at>2015-06-10 02:13:11 +0200
commit0888e7110e7ac23cb884d5dcdebe9090f02548ea (patch)
tree1ac5519036854762a7ef87e58e42d4b99efdf987
parent990c2ee3bd950f3b7d447cd71fa5efc979d1c98a (diff)
downloadffmpeg-0888e7110e7ac23cb884d5dcdebe9090f02548ea.tar.gz
avcodec/shorten: Fix code depending on signed overflow behavior
Signed-off-by: Michael Niedermayer <michaelni@gmx.at> (cherry picked from commit 2d15588124ab1d4c0612cab66f02a716f1509211) Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavcodec/shorten.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/libavcodec/shorten.c b/libavcodec/shorten.c
index 3bba10c398..08ea2178a4 100644
--- a/libavcodec/shorten.c
+++ b/libavcodec/shorten.c
@@ -131,8 +131,7 @@ static int allocate_buffers(ShortenContext *s)
av_log(s->avctx, AV_LOG_ERROR, "nmean too large\n");
return AVERROR_INVALIDDATA;
}
- if (s->blocksize + s->nwrap >= UINT_MAX / sizeof(int32_t) ||
- s->blocksize + s->nwrap <= (unsigned)s->nwrap) {
+ if (s->blocksize + (uint64_t)s->nwrap >= UINT_MAX / sizeof(int32_t)) {
av_log(s->avctx, AV_LOG_ERROR,
"s->blocksize + s->nwrap too large\n");
return AVERROR_INVALIDDATA;