Pavel Tisnovsky wrote:
Hi all,
I'd like to push fix for regression test java/io/IOException/LastErrorString.java 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: http://cr.openjdk.java.net/~ptisnovs/jtreg-jdk-test-LastErrorString-fix/
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? -Alan.