fergalmoran/ladybird
1
0
Fork 0
mirror of https://github.com/fergalmoran/ladybird.git synced 2026-02-12 02:36:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

LibGfx: Pad last element in ICC files to 4-byte boundary too

Browse Source 
The spec wants that to happen.
This commit is contained in:
Nico Weber
2023-02-23 09:10:26 -05:00
committed by Andreas Kling
parent 7853be7869
commit e8bd067ce5
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Userland/Libraries/LibGfx/ICC/BinaryWriter.cpp
View File

@@ -717,10 +717,13 @@ ErrorOr<ByteBuffer> encode(Profile const& profile)
offset += align_up_to(bytes.size(), 4);
}
// Omit padding after last element.
size_t total_size = offsets.last() + tag_data_bytes.last().size();
auto bytes = TRY(ByteBuffer::create_zeroed(total_size));
// Include padding after last element. Use create_zeroed() to fill padding bytes with null bytes.
// ICC v4, 7.1.2:
// "c) all tagged element data, including the last, shall be padded by no more than three following pad bytes to
// reach a 4-byte boundary;
// d) all pad bytes shall be NULL (as defined in ISO/IEC 646, character 0/0).
// NOTE 1 This implies that the length is required to be a multiple of four."
auto bytes = TRY(ByteBuffer::create_zeroed(offset));
for (size_t i = 0; i < tag_data_bytes.size(); ++i)
memcpy(bytes.data() + offsets[i], tag_data_bytes[i].data(), tag_data_bytes[i].size());