notgne2/tuxpaint-pencil-sharpener
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge 'Limit check to avoid crash in flood Fill' form 'master' into sdl2.0

Browse source
This commit is contained in:
Pere Pujal i Carabantes 2021-11-21 11:36:48 +01:00
commit 6007479b0e
1 changed files with 32 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files

64
src/fill.c
View file

@ -443,62 +443,62 @@ void simulate_flood_fill_outside_check(SDL_Surface * screen, SDL_Texture * textu
/* Continue filling upwards from this scanline */ /* Continue filling upwards from this scanline */
just_queued = 0; just_queued = 0;
for (i = narrowFillL; i <= narrowFillR; i++) if (y > 0)
{ {
px_colr = getpixels[global_last->format->BytesPerPixel] (global_last, i, y - 1); for (i = narrowFillL; i <= narrowFillR; i++)
closeness = colors_close(global_canvas, px_colr, global_old_colr);
if (y > 0 &&
(
closeness < COLOR_MATCH_NARROW ||
(closeness < COLOR_MATCH_WIDE && y_outside < WIDE_MATCH_THRESHOLD)
)
)
{ {
if (!just_queued && (global_touched == NULL || !global_touched[((y - 1) * global_canvas->w) + i])) px_colr = getpixels[global_last->format->BytesPerPixel] (global_last, i, y - 1);
closeness = colors_close(global_canvas, px_colr, global_old_colr);
if (closeness < COLOR_MATCH_NARROW ||
(closeness < COLOR_MATCH_WIDE && y_outside < WIDE_MATCH_THRESHOLD)
)
{ {
add_to_queue(i, y - 1, y_outside + 1); if (!just_queued && (global_touched == NULL || !global_touched[((y - 1) * global_canvas->w) + i]))
just_queued = 1; {
add_to_queue(i, y - 1, y_outside + 1);
just_queued = 1;
}
else
{
just_queued = 0;
}
} }
else else
{ {
just_queued = 0; just_queued = 0;
} }
} }
else
{
just_queued = 0;
}
} }
/* Continue filling downwards from this scanline */ /* Continue filling downwards from this scanline */
just_queued = 0; just_queued = 0;
for (i = narrowFillL; i <= narrowFillR; i++) if (y < global_canvas->h - 1)
{ {
px_colr = getpixels[global_last->format->BytesPerPixel] (global_last, i, y + 1); for (i = narrowFillL; i <= narrowFillR; i++)
closeness = colors_close(global_canvas, px_colr, global_old_colr);
if (y < global_canvas->h &&
(
closeness < COLOR_MATCH_NARROW ||
(closeness < COLOR_MATCH_WIDE && y_outside < WIDE_MATCH_THRESHOLD)
)
)
{ {
if (!just_queued && (global_touched == NULL || !global_touched[((y + 1) * global_canvas->w) + i])) px_colr = getpixels[global_last->format->BytesPerPixel] (global_last, i, y + 1);
closeness = colors_close(global_canvas, px_colr, global_old_colr);
if (closeness < COLOR_MATCH_NARROW ||
(closeness < COLOR_MATCH_WIDE && y_outside < WIDE_MATCH_THRESHOLD)
)
{ {
add_to_queue(i, y + 1, y_outside + 1); if (!just_queued && (global_touched == NULL || !global_touched[((y + 1) * global_canvas->w) + i]))
just_queued = 1; {
add_to_queue(i, y + 1, y_outside + 1);
just_queued = 1;
}
else
{
just_queued = 0;
}
} }
else else
{ {
just_queued = 0; just_queued = 0;
} }
} }
else
{
just_queued = 0;
}
} }
} }