Hello Brian, Thanks for the review. GIF stream in regression test case would match warn.gif stream behaviour that’s why it going into GIFImageReader.getCode().
Are you okay with overall webrev.00 patch or have you just approved GIFImageReader change? Please clarify. Regards, Jay > On 05-Mar-2020, at 10:20 PM, Brian Burkhalter <brian.burkhal...@oracle.com> > wrote: > > Hello Jay, > > The source fix looks OK to me. I get the same exception as in the bug > description when I run the test against my unpatched local JDK 15 build. > > Thanks, > > Brian > >> On Mar 5, 2020, at 2:12 AM, Jayathirth D v <jayathirth....@oracle.com >> <mailto:jayathirth....@oracle.com>> wrote: >> >> Please review the following fix for JDK 15: >> >> Bug : https://bugs.openjdk.java.net/browse/JDK-6532025 >> <https://bugs.openjdk.java.net/browse/JDK-6532025> >> Webrev : http://cr.openjdk.java.net/~jdv/6532025/webrev.00/ >> <http://cr.openjdk.java.net/~jdv/6532025/webrev.00/> >> >> Root cause : When we have truncated GIF images, stream.read() returns -1 but >> GIFImageReader doesn’t handle such conditions properly and continues to read >> input stream data. >> Solution : Handle cases where we reach EOF and throw appropriate exception. >
