[12] 8206403: ByteArrayOutputStream hugeCapacity method can return invalid capacity

Brian Burkhalter brian.burkhalter at oracle.com
Mon Jul 23 21:49:21 UTC 2018


Hi Roger,

Updated version: http://cr.openjdk.java.net/~bpb/8206403/webrev.01/

On Jul 23, 2018, at 2:10 PM, Roger Riggs <Roger.Riggs at Oracle.com> wrote:

> You might want to add an @requires of 8Gb or whatever so the test only runs on a system it can succeed on.

Re-tested and changed to @requires 2g and dropped the @ignore.

> I don't see the @randomness in the test.  (Other than perhaps available heap).

That was vestigial from copying the header from elsewhere.

> ByteArrayOutputStream:121: A message indicating the nature of the error would be useful.

There was no message in the original file but I concur that one is better so I added one.

> In the test, I don't think you need to fill the array, writing 0 is just as good as 0xff.

Changed.

Thanks for the review.

Brian


More information about the core-libs-dev mailing list