RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 15:57:34 +00:00
Code Issues Activity Actions

LibGfx/GIF: Avoid a silly loop in GIF logical screen parsing

Browse source
This commit is contained in:
Andreas Kling 2024-01-05 11:08:12 +01:00
parent 182a2b0c3a
commit c4d75ac11a
1 changed files with 2 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
Userland/Libraries/LibGfx/ImageFormats/GIFLoader.cpp
View file

@ -250,11 +250,9 @@ static ErrorOr<void> load_header_and_logical_screen(GIFLoadingContext& context)
if (global_color_table_flag) {
u8 bits_per_pixel = (packed_fields & 7) + 1;
int color_map_entry_count = 1;
for (int i = 0; i < bits_per_pixel; ++i)
color_map_entry_count *= 2;
size_t color_map_entry_count = 1 << bits_per_pixel;
for (int i = 0; i < color_map_entry_count; ++i) {
for (size_t i = 0; i < color_map_entry_count; ++i) {
u8 r = TRY(context.stream.read_value<u8>());
u8 g = TRY(context.stream.read_value<u8>());
u8 b = TRY(context.stream.read_value<u8>());