Reviewer needed - fix for regression test LastErrorString

Alan Bateman Alan.Bateman at
Thu May 5 19:45:05 UTC 2011

Pavel Tisnovsky wrote:
> Hi all,
> I'd like to push fix for regression test
> java/io/IOException/ to OpenJDK7 and OpenJDK6. This
> fix ensures that no system-important files can be erased and/or
> rewritten by this test even if root started JTreg. Fixed test also tries
> all various combinations of file attributes.
> Webrev is available at:
> Can anybody please review it?
> Thank you in advance,
> Pavel Tisnovsky
I agree that this test should be fixed but I don't know about jdk7 as 
it's being stabilized now and only critical changes are allowed.

 From a quick glance it looks like you've added the @test tag to both 
files so I assume jtreg will want to run it twice. Have you tried the 
test on Windows? I assume it will at least fail with "Unrecognized 
system" if run with Cygwin. In general it's best to avoid scripts if you 
can - in this case have you considered created an unreadable and 
unwriteable file in java instead?


More information about the core-libs-dev mailing list