Test: regression for signed atoi() -> unsigned wrap in IccTagXml ParseXml (#1346)

[colourbill-ctrl]

Summary

Deterministic regression for the #1346 sweep (fix: #1349). Replays a PoC profile with a negative cicpFields ColorPrimaries attribute through iccFromXml; under the integer / implicit-conversion sanitizer it fails if UBSan's implicit-integer-sign-change diagnostic reappears, and skips cleanly on a plain build.

Files

• .github/ci/test-data/ub-cicp-colorprimaries-1346.xml — PoC: a valid copy of Testing/HDR/BT2100PQFullScene.xml with ColorPrimaries="-1". Because it is otherwise valid, iccFromXml parses and saves it; the pass/fail signal is purely the sanitizer diagnostic, not the tool exit code.
• .github/scripts/iccdev-issue-1346-imageparse-atoi-unsigned-regression.sh — gates on the build's CMakeCache.txt (ENABLE_INTEGER_SANITIZER/ENABLE_UBSAN, or an explicit -fsanitize=*integer/undefined/implicit); [SKIP]s otherwise; signal = presence of runtime error: implicit conversion. Exit 0 pass/skip, 2 on finding.
• Build/Cmake/Testing/CMakeLists.txt — registers iccdev.issue-1346-imageparse-atoi-unsigned-regression (labels include ubsan).

Scope note

This exercises the implicit Class A arm of the sweep. The explicit-cast Class B sites carry an (icUIntNN) cast that suppresses the UBSan check, so they are invisible to any sanitizer-based test by construction — they are guarded by the shared icXmlAttrToUInt() helper and review in #1349, not here.

Verification (clang-18, -DENABLE_ASAN=ON -DENABLE_UBSAN=ON -DENABLE_INTEGER_SANITIZER=ON)

• pre-fix master: [FAIL] exit 2 — IccTagXml.cpp:1172: runtime error: implicit conversion from type 'int' of value -1 ... changed the value to 255, stack in CIccTagXmlCicp::ParseXml.
• with Fix: signed atoi() -> unsigned wrap across IccTagXml ParseXml handlers (#1346) #1349: [PASS] exit 0.
• plain build: [SKIP].

⚠️ Expected to stay red on master until the fix #1349 is merged — the regression is doing its job by detecting the unfixed UB. Merge #1349 first, then this goes green (fork-gate split: fix on fork, test upstream).

🤖 Generated with Claude Code

[xsscx]

2026-06-14 20:01:37 UTC

[colourbill-ctrl]

@xsscx note on the red CTest ASAN+UBSAN check here:

Expected red-by-design — the paired fix (#1349) is not merged yet, so master still has the signed atoi()→unsigned wrap and the regression correctly fails on it (UBSan implicit conversion at IccTagXml.cpp:1172, CIccTagXmlCicp::ParseXml).

Sequence to green:

1. merge fix Fix: signed atoi() -> unsigned wrap across IccTagXml ParseXml handlers (#1346) #1349
2. I refresh this branch with master
3. CTest goes green → merge this test PR

Verified locally: FAILS on pre-fix master, PASSES with #1349, SKIPS on a non-sanitizer build.