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

[parser] Turn illegal item header parse into recoverable error #548

Merged
merged 1 commit into from
Jul 28, 2024

Conversation

Enet4
Copy link
Owner

@Enet4 Enet4 commented Jul 28, 2024

Fixes panic caused by an item start after an extraneous item end. Resolves #547.

- fixes panic caused by an item start after an extraneous item end
- add test case covering this particular scenario
@Enet4 Enet4 added bug This is a bug A-lib Area: library C-parser Crate: dicom-parser labels Jul 28, 2024
@Enet4 Enet4 merged commit af18fbe into master Jul 28, 2024
4 checks passed
@Enet4 Enet4 deleted the bug/parser/547-item-header-out-sequence branch July 28, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-lib Area: library bug This is a bug C-parser Crate: dicom-parser
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Panic - item header should be read only inside an existing sequence
1 participant