2017-10-30 10:14 GMT+01:00 Nicolas George <geo...@nsup.org>:
> Move a subtraction to the other side of the equal to avoid overflows.
>
> Signed-off-by: Nicolas George <geo...@nsup.org>
> ---
> libavfilter/vsrc_testsrc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavfilter/vsrc_testsrc.c b/libavfilter/vsrc_testsrc.c
> index fe0d50aa41..a790974d14 100644
> --- a/libavfilter/vsrc_testsrc.c
> +++ b/libavfilter/vsrc_testsrc.c
> @@ -857,8 +857,8 @@ static void test2_fill_picture(AVFilterContext *ctx,
> AVFrame *frame)
> uint8_t alpha[256];
>
> r = s->pts;
> - for (y = ymin; y < ymax - 15; y += 16) {
> - for (x = xmin; x < xmax - 15; x += 16) {
> + for (y = ymin; y + 15 < ymax; y += 16) {
> + for (x = xmin; x + 15 < xmax; x += 16) {
> if ((x ^ y) & 16)
> continue;
> for (i = 0; i < 256; i++) {
> --
> 2.14.2
lgtm
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
