[FFmpeg-cvslog] swresample/resample: Limit filter length
Michael Niedermayer
git at videolan.org
Tue Apr 22 00:59:47 CEST 2014
ffmpeg | branch: release/1.1 | Michael Niedermayer <michaelni at gmx.at> | Tue Apr 8 00:19:07 2014 +0200| [0143eb91098f7de89ba5f6885a91f0aa3e6e34ed] | committer: Michael Niedermayer
swresample/resample: Limit filter length
Related to CID1197063
The limit choosen is arbitrary and much larger than what makes sense.
It avoids the need for checking arithmetic operations with the length for overflow
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
(cherry picked from commit f9158b01d0f3effb58e87fb07db0382bc1e47de5)
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=0143eb91098f7de89ba5f6885a91f0aa3e6e34ed
---
libswresample/resample.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/libswresample/resample.c b/libswresample/resample.c
index 6cd2b8c..96bc921 100644
--- a/libswresample/resample.c
+++ b/libswresample/resample.c
@@ -229,6 +229,11 @@ static ResampleContext *resample_init(ResampleContext *c, int out_rate, int in_r
av_assert0(0);
}
+ if (filter_size/factor > INT32_MAX/256) {
+ av_log(NULL, AV_LOG_ERROR, "Filter length too large\n");
+ goto error;
+ }
+
c->phase_shift = phase_shift;
c->phase_mask = phase_count - 1;
c->linear = linear;
More information about the ffmpeg-cvslog
mailing list