Skip to content

Fix: CIccCLUT::Iterate() & CIccMBB::Describe()#737

Merged
xsscx merged 1 commit intomasterfrom
issue-704
Mar 27, 2026
Merged

Fix: CIccCLUT::Iterate() & CIccMBB::Describe()#737
xsscx merged 1 commit intomasterfrom
issue-704

Conversation

@xsscx
Copy link
Copy Markdown
Member

@xsscx xsscx commented Mar 27, 2026

Pull Request Checklist

(#704)

  • Have you followed the guidelines in Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you built your Pull Request locally with the Build Instructions?
  • Have you added or updated relevant tests?
  • Have you added or updated relevant docs?

@xsscx xsscx requested a review from ChrisCoxArt March 27, 2026 04:01
@xsscx xsscx self-assigned this Mar 27, 2026
@xsscx xsscx added PR Pull Request Merge Ready Maintainer indicates Merge Ready labels Mar 27, 2026
@xsscx xsscx changed the title Fix: CIccCLUT::Iterate() Fix: CIccCLUT::Iterate() & CIccMBB::Describe() Mar 27, 2026
@xsscx xsscx added Pending Merge Maintainer indicates Merge Pending and requests no further changes and removed Merge Ready Maintainer indicates Merge Ready pending labels Mar 27, 2026
@xsscx
Copy link
Copy Markdown
Member Author

xsscx commented Mar 27, 2026

PR Author Summary

2026-03-27 04:34:56 UTC

This PR addresses #704 & #705

Unpatched Outputs

BEGIN_LUT CLUT 3 4
 CMYK_1=3 CMYK_2=3 CMYK_3=3   Lab_1 Lab_2 Lab_3 Lab_4
BEGIN_LUT A_Curve_Lab_1 1 1
IN OUT

Patched Outputs

[2026-03-27 04:34:56 UTC] ~/issue-704/iccDEV/Testing (issue-704)$ git log --oneline --graph -1
* b606697 (HEAD -> issue-704, origin/issue-704) Fix: CIccCLUT::Validate()
[2026-03-27 04:35:25 UTC] ~/issue-704/iccDEV/Testing (issue-704)$ iccDumpProfile -v 100 cfl-049-btoa-lut16-test.icc ALL 2>/dev/null | grep -A3 "BEGIN_LUT CLUT"
BEGIN_LUT CLUT 3 4
 CMYK_1=3 CMYK_2=3 CMYK_3=3   Lab_1 Lab_2 Lab_3 Lab_4
   0.000   0.000   0.000     0.000 -125.609 -123.219   2.814
   0.000   0.000  50.000     3.752 -116.039 -113.648   6.567
--
BEGIN_LUT CLUT 4 3
 Lab_1=3 Lab_2=3 Lab_3=3 Lab_4=3   CMYK_1 CMYK_2 CMYK_3
   0.000 -128.000 -128.000   0.000    18.837  18.894  18.950
   0.000 -128.000 -128.000  50.195    19.007  19.063  19.120

@xsscx xsscx merged commit 6e8a618 into master Mar 27, 2026
24 checks passed
@xsscx xsscx removed the Pending Merge Maintainer indicates Merge Pending and requests no further changes label Mar 27, 2026
@xsscx xsscx deleted the issue-704 branch March 27, 2026 18:33
@xsscx xsscx added the CVE Requested Maintainer indicates a CVE has been Requested label Mar 28, 2026
@xsscx
Copy link
Copy Markdown
Member Author

xsscx commented Mar 28, 2026

GHSA-5r4q-77w5-3q3h

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CVE Requested Maintainer indicates a CVE has been Requested PR Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Patch: CIccMBB::Describe() in IccTagLut.cpp for dumpProfile-LUT Patch: DumpLut Iterate() in IccTagLut.cpp in dumpProfile-LUT

2 participants