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

Palette image with cdef fails to decompress #412

Closed
gcode-importer opened this issue Oct 10, 2014 · 2 comments
Closed

Palette image with cdef fails to decompress #412

gcode-importer opened this issue Oct 10, 2014 · 2 comments
Assignees

Comments

@gcode-importer
Copy link

Originally reported on Google Code with ID 412

Image with no cdef decodes properly
./bin/opj_decompress -i palette.jp2 -o palette.bmp 

[INFO] Start to read j2k main header (913).
[INFO] Main header has been correctly decoded.
[INFO] No decoded area parameters, set the decoded area to the whole image
[INFO] Header of tile 1 / 1 has been read.
[INFO] Tile 1/1 has been decoded.
[INFO] Image data has been updated with tile 1.

[INFO] Stream reached its end !
[INFO] Generated Outfile 0.bmp

With cdef, not decoded :
./bin/opj_decompress -i palette-cdef.jp2 -o palette-cdef.bmp 

[INFO] Start to read j2k main header (941).
[INFO] Main header has been correctly decoded.
[INFO] No decoded area parameters, set the decoded area to the whole image
[INFO] Header of tile 1 / 1 has been read.
[INFO] Tile 1/1 has been decoded.
[INFO] Image data has been updated with tile 1.

[ERROR] Invalid component index 1 (>= 1).
ERROR -> opj_decompress: failed to decode image!

cdef box was inserted between the 2 to rotate channels
Patch is ready but waiting for other commits as it would conflict.
Kakadu decodes this properly

Reported by mayeut on 2014-10-10 22:47:18


- _Attachment: [palette.jp2](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-412/comment-0/palette.jp2)_ - _Attachment: [palette-cdef.jp2](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-412/comment-0/palette-cdef.jp2)_ - _Attachment: [palette.bmp](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-412/comment-0/palette.bmp)_ - _Attachment: [palette-cdef.bmp](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-412/comment-0/palette-cdef.bmp)_
@gcode-importer
Copy link
Author

Patch verified against the test suite & ok

Reported by mayeut on 2014-10-23 19:45:13

  • Status changed: Verified

- _Attachment: [issue412-r2918.patch](https://storage.googleapis.com/google-code-attachments/openjpeg/issue-412/comment-1/issue412-r2918.patch)_

@gcode-importer
Copy link
Author

This issue was closed by revision r2922.

Reported by detonin on 2014-10-30 18:26:13

  • Status changed: Fixed

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

2 participants