Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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.

Sign up for GitHub

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

Cannot read RLE Lossless DICOM images #6566

Closed
jochalek opened this issue Aug 16, 2023 · 2 comments
Closed

Cannot read RLE Lossless DICOM images #6566

jochalek opened this issue Aug 16, 2023 · 2 comments
Labels
up for grabs

Comments

@jochalek
Copy link

ImageMagick version

7.1.1-15

Operating system

Linux

Operating system, version and so on

Ubuntu 22.04.3 LTS

Description

Attempting to read RLE Lossless DICOM images with Transfer Syntax UID "1.2.840.10008.1.2.5" results in improper image header error.

I'm not certain what the problem is. My DICOM data are RLE Lossless encoded and cause this error. Any RLE lossless encoded samples I can find elsewhere also cause this error. DICOMs encoded in other formats do work.

I can reproduce the same problem with ImageMagick6 as well.

Steps to Reproduce

user@ubuntu:~/$ identify ~/Downloads/US-PAL-8-10x-echo
identify: improper image header `/home/user/Downloads/US-PAL-8-10x-echo' @ error/dcm.c/ReadDCMImage/3210.

Images

The example image may be found here US-PAL-8-10x-echo.
@dlemstra
Copy link
Member

It looks like this image contains something we don't support or we are reading the RLE compression incorrectly. Someone will need to investigate that.

Adding the file here for future reference: US-PAL-8-10x-echo.gz

@urban-warrior
Copy link
Member

Thanks for the problem report. We can reproduce the issue and will have a patch to fix it in the GIT main branch @ https://github.com/ImageMagick/ImageMagick later today. The patch will be available in the beta releases of ImageMagick @ https://imagemagick.org/archive/beta/ by sometime tomorrow.

urban-warrior pushed a commit to ImageMagick/ImageMagick6 that referenced this issue Sep 3, 2023
account for extra bytes at end of a DICOM image (ImageMagick/ImageMag…
093e1db 
…ick#6566)
urban-warrior pushed a commit that referenced this issue Sep 3, 2023
account for extra bytes at end of a DICOM image (#6566)
4cf2892
@ImageMagick ImageMagick locked and limited conversation to collaborators Sep 22, 2023
@urban-warrior urban-warrior converted this issue into discussion #6669 Sep 22, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Assignees
No one assigned
Labels
up for grabs
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dlemstra @urban-warrior @jochalek