On Tue, 2 May 2023 15:52:03 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>> Volker Simonis has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Throw StreamCorruptedException instead of InvalidClassException and handle 
>> negative array size in checkArray() as well
>
> test/jdk/java/io/ObjectInputStream/NegativeArraySizeTest.java line 77:
> 
>> 75:        byte[] serializedData = baos.toByteArray();
>> 76: 
>> 77:        // Find the right location to modify, looking for the first 
>> instance of TC_ENDBLOCKDATA
> 
> The comment should probably mention TC_BLOCKDATA, not TC_END...

Doh! Fixed two comments but forgot the third one. Will fix.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13540#discussion_r1182803335

Reply via email to