https://bugs.kde.org/show_bug.cgi?id=496350

freshly...@mailinator.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---
                 CC|                            |freshly...@mailinator.com

--- Comment #9 from freshly...@mailinator.com ---
Some animated JXLs still fail to decode with kimageformats 6.9.0:

`Unexpected event 16384 instead of JXL_DEC_FRAME`

(16384 = JXL_DEC_BOX)

The code around
https://invent.kde.org/frameworks/kimageformats/-/blob/92a1752c1f72543b6693ee5113b5769b38b5edf4/src/imageformats/jxl.cpp#L361
assumes that all the JXL_DEC_FRAME events are contiguous, but in fact they can
interleave with JXL_DEC_BOX.

I don't have the example file to share at the moment, but the sequence of
decoder events was as follows:

JXL_DEC_BOX
JXL_DEC_BOX
JXL_DEC_BOX
JXL_DEC_BASIC_INFO
JXL_DEC_COLOR_ENCODING
JXL_DEC_BOX
JXL_DEC_BOX
JXL_DEC_FRAME
JXL_DEC_BOX
JXL_DEC_FRAME

Notice the FRAME - BOX - FRAME sequence at the end, which confuses the
`countALLFrames` function.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to