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

#834 Support skipping input chunks after a buffer underflow #850

Merged
merged 1 commit into from
Aug 4, 2021

Conversation

theigl
Copy link
Collaborator

@theigl theigl commented Aug 4, 2021

Buffer underflows set the buffer position to 0 while trying to resize the buffer. This prevents InputChunked.nextChunk() from advancing to the next chunk.

This PR ensures that that the buffer is at the final position of the chunk before advancing to the next chunk.

Resolves #834.

@theigl theigl added the bug label Aug 4, 2021
@theigl theigl self-assigned this Aug 4, 2021
@theigl theigl force-pushed the 834-skip-chunk-after-underflow branch from b093eca to fd6e657 Compare August 4, 2021 18:01
@theigl theigl merged commit 8785e6e into master Aug 4, 2021
@theigl theigl deleted the 834-skip-chunk-after-underflow branch August 4, 2021 18:09
theigl added a commit that referenced this pull request May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

Buffer underflow while deserializing with Collection type field removed (with reproducible test case)
1 participant