Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move bracket depth checking into parsers. #101

Merged
merged 1 commit into from Apr 3, 2020
Merged

Move bracket depth checking into parsers. #101

merged 1 commit into from Apr 3, 2020

Conversation

kandykoma
Copy link
Contributor

@kandykoma kandykoma commented Apr 2, 2020 •
edited

Improves #97, this way the checking also applies to filtered (compressed) data.

base64 encoded sample pdf:

JVBERi0xLjUKMSAwIG9iajw8L1R5cGUvUGFnZXMvS2lkc1s1IDAgUl0vQ291bnQgMS9SZXNvdXJj
ZXMgMyAwIFIvTWVkaWFCb3hbMCAwIDU5NSA4NDJdPj5lbmRvYmoKMiAwIG9iajw8L1R5cGUvRm9u
dC9TdWJ0eXBlL1R5cGUxL0Jhc2VGb250L0NvdXJpZXI+PmVuZG9iagozIDAgb2JqPDwvRm9udDw8
L0YxIDIgMCBSPj4+PmVuZG9iago1IDAgb2JqPDwvVHlwZS9QYWdlL1BhcmVudCAxIDAgUi9Db250
ZW50c1s0IDAgUl0+PmVuZG9iago2IDAgb2JqPDwvVHlwZS9DYXRhbG9nL1BhZ2VzIDEgMCBSPj5l
bmRvYmoKNCAwIG9iajw8L0xlbmd0aCAgIDg0IC9GaWx0ZXIvRmxhdGVEZWNvZGU+PnN0cmVhbQp4
2u3DsQmDUBAA0P6muHRJFYUQUgcSHODABdRCPgjuX+gSdu/B+1Y8/32+PllL9F2X73NNcQcAAAAA
LjfMrW05bnubbg8AAAAA4HJZa/wqDnT6OQplbmRzdHJlYW0gZW5kb2JqCnhyZWYKMCA3CjAwMDAw
MDAwMDAgNjU1MzUgZiAKMDAwMDAwMDAwOSAwMDAwMCBuIAowMDAwMDAwMDk2IDAwMDAwIG4gCjAw
MDAwMDAxNTUgMDAwMDAgbiAKMDAwMDAwMDI5MSAwMDAwMCBuIAowMDAwMDAwMTkxIDAwMDAwIG4g
CjAwMDAwMDAyNDggMDAwMDAgbiAKdHJhaWxlcgo8PC9Sb290IDYgMCBSL1NpemUgNz4+CnN0YXJ0
eHJlZgo0NDIKJSVFT0YK

@kandykoma
Move bracket depth checking into parsers.
1168f89 
This way the checking also applies to filtered (compressed) data.
@J-F-Liu J-F-Liu merged commit 5e53752 into J-F-Liu:master Apr 3, 2020
@kandykoma kandykoma mentioned this pull request Apr 4, 2020
Merged
@kandykoma kandykoma mentioned this pull request Apr 22, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kandykoma @J-F-Liu