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

LibPDF: Handle pdf-specific white spaces correctly in ASCII85

Browse source 
We were previously only looking the space character but PDF white
spaces is a superset of ascii spaces.
This commit is contained in:
Lucas CHOLLET 2023-11-11 17:38:53 -05:00 • committed by Andreas Kling
parent db08fe12ec
commit 2fe0647c68
1 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
Userland/Libraries/LibPDF/Filter.cpp
View file

@ -11,6 +11,7 @@
#include <LibGfx/ImageFormats/JPEGLoader.h>
#include <LibPDF/CommonNames.h>
#include <LibPDF/Filter.h>
#include <LibPDF/Reader.h>
namespace PDF {
@ -99,7 +100,7 @@ PDFErrorOr<ByteBuffer> Filter::decode_ascii85(ReadonlyBytes bytes)
size_t byte_index = 0;
while (byte_index < bytes.size()) {
if (bytes[byte_index] == ' ') {
if (Reader::is_whitespace(bytes[byte_index])) {
byte_index++;
continue;
}
@ -117,7 +118,7 @@ PDFErrorOr<ByteBuffer> Filter::decode_ascii85(ReadonlyBytes bytes)
auto to_write = bytes.size() - byte_index;
for (int i = 0; i < 5; i++) {
auto byte = byte_index >= bytes.size() ? 'u' : bytes[byte_index++];
if (byte == ' ') {
if (Reader::is_whitespace(byte)) {
i--;
continue;
}
@ -131,7 +132,7 @@ PDFErrorOr<ByteBuffer> Filter::decode_ascii85(ReadonlyBytes bytes)
} else {
for (int i = 0; i < 5; i++) {
auto byte = bytes[byte_index++];
if (byte == ' ') {
if (Reader::is_whitespace(byte)) {
i--;
continue;
}