RFR [7129312] BufferedInputStream calculates negative array size with large streams and mark

Alan Bateman Alan.Bateman at oracle.com
Wed Aug 21 09:28:48 UTC 2013


On 20/08/2013 17:01, Ivan Gerasimov wrote:
> Yes, you're right!
>
> I've updated the test to be a shell script + Java application.
> If java reported 'Invalid maximum heap size', the test is skipped.
>
> Just check with JPRT, the test passes on both 64 and 32 bit platforms.
>
> Would you please review the updated webrev?
> http://cr.openjdk.java.net/~igerasim/7129312/2/webrev/ 
> <http://cr.openjdk.java.net/%7Eigerasim/7129312/2/webrev/>
The updated change to BufferedInputStream looks good.

On the test then I agree with the approach to skip on 32-bit but I think 
we should avoid introduce more shell tests. Have you looked at 
jdk.testlibrary.ProcessTools to launch the test after checking that we 
are 64-bit?

-Alan.




More information about the core-libs-dev mailing list