Hi Sean, I might need to make a change but this is the JTREG output from my local run jtreg -jdk:./build/windows-x86_64-normal-server-release/images/jdk ./jdk/test/java/lang/SecurityManager/
Execution successful java/lang/SecurityManager/CheckPackageAccess.java: Make sure all restricted packages listed in the package.access property in the java.security file are blocked java/lang/SecurityManager/CheckPackageMatching.java: Check the matching implemented by SecurityManager.checkPackageAccess java/lang/SecurityManager/CheckSecurityProvider.java: Consolidate java.security files into one file with modifications java/lang/SecurityManager/NoAWT.java: Sanity check that SecurityManager methods that used to check AWTPermission now check for AllPermission so it looks like there are no problems. What do you think? I'll report back once my JPRT run finishes. Pete On 8/27/15 4:50 PM, Sean Mullan wrote: > This looks good to me, if the JPRT results are good. However, does > this also fix the failure in CheckPackageMatching? I would have > expected you needed an additional fix - see my subsequent comment: > > https://bugs.openjdk.java.net/browse/JDK-8134456?focusedCommentId=13837407&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13837407 > > > Thanks, > Sean > > On 08/27/2015 05:02 PM, Pete Brunet wrote: >> Please review a patch for a test change which is needed to fix an issue >> caused by JDK-8051626. Tier 1 tests are being impacted. >> >> http://cr.openjdk.java.net/~ptbrunet/JDK-8134456/webrev.00/ >> >> Pete >>
