RFR: 8160327: Support for thumbnails present in APP1 marker for JPEG [v7]
Jeremy
duke at openjdk.org
Sat Mar 1 18:01:05 UTC 2025
On Sat, 1 Mar 2025 17:46:10 GMT, Harald Kuhr <duke at openjdk.org> wrote:
>> Jeremy has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8160327: trying to placate PR script
>>
>> The github script still classifies two of the sample jpgs as executable files, which it classifies as errors.
>
> src/java.desktop/share/classes/com/sun/imageio/plugins/jpeg/ExifMarkerSegment.java line 182:
>
>> 180: // file shows it can also sometimes be 0x60000. I've also observed it to be
>> 181: // undefined, 0x0007, or several variations of 0x????0006. Similarly the same
>> 182: // tag should be 0x0001 for TIFFs, but I also observed a case where it as 0x10000.
>
> Isn't this ( 0x0001/0x0006 vs 0x1000/0x6000) just a matter of endianness in the TIFF structure? Some odd writers may also use LONG/32 bit values, even though the TiFF and Exif specs only mention SHORT/16 bit values for the compression tag.
>
> Compression 7 "New JPEG" is not as per the Exif spec, but it can probably safely be treated the same way as "Old JPEG" compression 6 for Exif thumbnails.
Yes, it probably is endianness, or endianness-related. My first design question is: should we care? Currently this PR infers whether we're looking for a JPEG or TIFF thumbnail based on other fields. If we strictly rely on the compression tag (250) instead: is that better/desirable? (That is: we could just throw an IOException in the rare case this field is missing/broken.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22898#discussion_r1976466185
More information about the client-libs-dev
mailing list