On Tue, 2 May 2023 16:59:00 GMT, Aleksey Shipilev <sh...@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 106:
>
>> 104:
>> 105: public static void main(String[] args) throws Exception {
>> 106: try {
>
> Test code nits, we can probably `try-with-resources` this thing to make it
> cleaner and make sure nothing unexpectedly bad is happening on "close" paths.
OK, here we go...
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13540#discussion_r1182871096
