If less than 2 bytes left, there will not be another SOT or an EOC

[radarhere]

No description provided.

[rouault]

@radarhere Can you attach a JPEG2000 file demonstrating the need for this change ?

[radarhere]

Oh, I didn't mean to imply there was a need. I just considered opj_j2k_read_sod in isolation, and thought that if there were less than 2 bytes left, then

openjpeg/src/lib/openjp2/j2k.h

Line 130 in eb25a5e

J2K_STATE_TPHSOT = 0x0008, /**< the decoding process is in a tile part header and expects a SOT marker */

wasn't the case, and

openjpeg/src/lib/openjp2/j2k.h

Line 133 in eb25a5e

J2K_STATE_NEOC = 0x0040, /**< the decoding process must not expect a EOC marker because the codestream is truncated */

reflected reality better.

If you wish to decline this for the sake of performance or for the sake of code simplicity, that is fine.

[rouault]

IMHO this code base is sufficiently tricky to avoid adding new code without a clear evidence it is needed

[radarhere]

Ok, sure. Thanks for the review.