[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: [OpenJDK 2D-Dev] [16] RFR JDK-8243674: Remove language tag length limit for iTXt chunk in PNGIma
From: Sergey Bylokhov <Sergey.Bylokhov () oracle ! com>
Date: 2020-07-31 20:15:03
Message-ID: 062efcd4-adb9-6dab-9a1a-6a3a789791b0 () oracle ! com
[Download RAW message or body]
ok, +1
On 31.07.2020 07:21, Jayathirth D v wrote:
> Thanks for the review Sergey.
> The extra check for length will hit only in case of corrupted chunk. Since we don't \
> know the exact length of "languageTag" or "Translated keyword" or "Text" we use \
> chunkLength to calculate remaining length. This fix is to comply with spec. \
> Testcase throws appropriate "Found non null terminated string" IIOException without \
> this fix(because languageTag can be more than 80) and works properly with fix.
> Regards,
> Jay
>
> > On 31-Jul-2020, at 4:25 AM, Sergey Bylokhov <Sergey.Bylokhov@oracle.com> wrote:
> >
> > Hi, Jay.
> >
> > The fix looks fine, but maybe we can to trigger added IIOException(remainingLen < \
> > 0) by the test?
> > On 29.07.2020 02:41, Jayathirth D v wrote:
> > > Hello All,
> > > Please review the following fix for JDK 16:
> > > Bug : https://bugs.openjdk.java.net/browse/JDK-8243674
> > > Webrev : http://cr.openjdk.java.net/~jdv/8243674/webrev.00/
> > > Issue : We have language tag length limit of 80 for iTXt chunk in \
> > > PNGImageReader which is not spec \
> > > compliant(http://www.libpng.org/pub/png/spec/1.2/PNG-Chunks.html#C.iTXt). There \
> > > should not be limit on length of language tag. Solution : Remove language tag \
> > > restriction of 80. In PNGImageWriter we don't enforce any limit on language tag \
> > > length. Thanks,
> > > Jay
> >
> >
> > --
> > Best regards, Sergey.
>
--
Best regards, Sergey.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic