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

Should the block padding size validated? #16

Open
ulikunitz opened this issue Jun 5, 2017 · 0 comments
Open

Should the block padding size validated? #16

ulikunitz opened this issue Jun 5, 2017 · 0 comments
Assignees
Milestone

Comments

@ulikunitz
Copy link
Owner

Thanks to Dórian C. Langbeck I realized that I confused the padding for the block header with the padding of the block in the discussion of issue #15 . Currently we don't check the block padding size. Whether we should check it is an open question.

3.3. Block Padding

        Block Padding MUST contain 0-3 null bytes to make the size of
        the Block a multiple of four bytes. This can be needed when
        the size of Compressed Data is not a multiple of four. If any
        of the bytes in Block Padding are not null bytes, the decoder
        MUST indicate an error.
@ulikunitz ulikunitz added this to the v0.6 milestone Jun 5, 2017
@ulikunitz ulikunitz self-assigned this Jun 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant