Skip to content

BUG: Fix wrong LUT size error#3651

Merged
stefan6419846 merged 2 commits intopy-pdf:mainfrom
stefan6419846:image-lut
Feb 19, 2026
Merged

BUG: Fix wrong LUT size error#3651
stefan6419846 merged 2 commits intopy-pdf:mainfrom
stefan6419846:image-lut

Conversation

@stefan6419846
Copy link
Collaborator

Running the image extraction on an image with DeviceCMYK and JPXDecode using a custom /Decode entry would previously fail when applying the LUT to the image in _apply_decode. This was due to converting the image to RGB, but the custom LUT having a size of 8 entries.

For my specific (non-disclosable) example everything works correctly now, while all existing tests still pass. Honestly, I am not sure whether all cases are handled correctly now, but at least the error is gone ...

Running the image extraction on an image with DeviceCMYK and JPXDecode
using a custom /Decode entry would previously fail when applying the LUT
to the image in `_apply_decode`. This was due to converting the image to
RGB, but the custom LUT having a size of 8 entries.

For my specific (non-disclosable) example everything works correctly
now, while all existing tests still pass. Honestly, I am not sure
whether all cases are handled correctly now, but at least the error is
gone ...
@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

❌ Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 97.35%. Comparing base (801353a) to head (b579926).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pypdf/_xobj_image_helpers.py 88.88% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3651      +/-   ##
==========================================
- Coverage   97.37%   97.35%   -0.02%     
==========================================
  Files          55       55              
  Lines        9909     9916       +7     
  Branches     1811     1814       +3     
==========================================
+ Hits         9649     9654       +5     
- Misses        151      152       +1     
- Partials      109      110       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stefan6419846 stefan6419846 merged commit 4670513 into py-pdf:main Feb 19, 2026
16 of 18 checks passed
@stefan6419846 stefan6419846 deleted the image-lut branch February 19, 2026 13:38
stefan6419846 added a commit that referenced this pull request Feb 22, 2026
## What's new

### Security (SEC)
- Prevent infinite loop from circular xref /Prev references (#3655) by @rampageservices

### Bug Fixes (BUG)
- Fix wrong LUT size error (#3651) by @stefan6419846
- Fix handling of page boxes defined on `/Pages` (#3650) by @stefan6419846

[Full Changelog](6.7.1...6.7.2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant