RFR: 8266435: WBMPImageReader.read() should not truncate the input stream

[Sergey Bylokhov]

On Tue, 23 Nov 2021 11:05:27 GMT, Jayathirth D V <jdv at openjdk.org> wrote:

> If we use a custom stream and specify limit on stream.read() length, WBMPImageReader.read() doesnt verify whether we are decoded complete data or not. We can check the length of data decoded and rerun the stream.read() or use readFully(). In case of other decoders like BMP we are using readFully(), so i have updated WBMPImageReader.read() to use readFully().

test/jdk/javax/imageio/plugins/wbmp/WBMPStreamTruncateTest.java line 61:

> 59:         File imageFile = File.
> 60:                 createTempFile("test", ".wbmp", new File(filePath));
> 61:         imageFile.deleteOnExit();

In one another review I saw the deleteOnExit() usage, I remember that in case of full testrun via makefile, such files were not deleted, can you please confirm that it is work fine? probably it that was fixed already. Note that the full test run uses custom tmp folder inside the result dir.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6518