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

[ts-registry] constrain number of frames decoded and increase resilience in the JPEG adapter #535

Merged
merged 2 commits into from
Jul 13, 2024

Conversation

Enet4
Copy link
Owner

@Enet4 Enet4 commented Jul 9, 2024

Do not try to decode more frames than it should.

- do not try to decode more frames than it should
@Enet4 Enet4 added bug This is a bug A-lib Area: library C-transfer-syntax Crate: dicom-transfer-syntax-registry jpeg Support for the JPEG family of transfer syntaxes labels Jul 9, 2024
@Enet4 Enet4 linked an issue Jul 11, 2024 that may be closed by this pull request
@Enet4 Enet4 changed the title [ts-registry] constrain number of frames decoded in the JPEG adapter [ts-registry] constrain number of frames decoded and increase resilience in the JPEG adapter Jul 13, 2024
- stop if there are only 2 bytes left
- replace loop count with for loop
@Enet4 Enet4 force-pushed the bug/ts-registry/jpeg-decode-constrain-frames branch from 10d0fea to d212b78 Compare July 13, 2024 14:19
@Enet4 Enet4 merged commit 239cba5 into master Jul 13, 2024
4 checks passed
@Enet4 Enet4 deleted the bug/ts-registry/jpeg-decode-constrain-frames branch July 13, 2024 14:29
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-transfer-syntax Crate: dicom-transfer-syntax-registry jpeg Support for the JPEG family of transfer syntaxes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

First two bytes are not an SOI marker
1 participant