test/jdk/java/lang/System/LoggerFinder/internal/BaseDefaultLoggerFinderTest/BaseDefaultLoggerFinderTest.java

rev 47439 : 8189291: Test policy should extend the default system policy
Reviewed-by:

*** 1,7 ****
  /*
!  * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
--- 1,7 ----
  /*
!  * Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
*** 56,66 ****
  import jdk.internal.logger.SimpleConsoleLogger;
  import sun.util.logging.PlatformLogger;
  
  /**
   * @test
!  * @bug     8140364 8145686
   * @summary JDK implementation specific unit test for the base DefaultLoggerFinder.
   *          Tests the behavior of DefaultLoggerFinder and SimpleConsoleLogger
   *          implementation.
   * @modules java.base/sun.util.logging
   *          java.base/jdk.internal.logger
--- 56,66 ----
  import jdk.internal.logger.SimpleConsoleLogger;
  import sun.util.logging.PlatformLogger;
  
  /**
   * @test
!  * @bug     8140364 8145686 8189291
   * @summary JDK implementation specific unit test for the base DefaultLoggerFinder.
   *          Tests the behavior of DefaultLoggerFinder and SimpleConsoleLogger
   *          implementation.
   * @modules java.base/sun.util.logging
   *          java.base/jdk.internal.logger
*** 73,82 ****
--- 73,83 ----
   * @run  main/othervm -Xbootclasspath/a:boot -Djava.system.class.loader=CustomSystemClassLoader BaseDefaultLoggerFinderTest WITHREFLECTION
   * @author danielfuchs
   */
  public class BaseDefaultLoggerFinderTest {
  
+     static final Policy DEFAULT_POLICY = Policy.getPolicy();
      static final RuntimePermission LOGGERFINDER_PERMISSION =
                  new RuntimePermission("loggerFinder");
      final static boolean VERBOSE = false;
      static final ThreadLocal<AtomicBoolean> allowControl = new ThreadLocal<AtomicBoolean>() {
          @Override
*** 957,967 ****
              return permissions;
          }
  
          @Override
          public boolean implies(ProtectionDomain domain, Permission permission) {
!             return getPermissions().implies(permission);
          }
  
          @Override
          public PermissionCollection getPermissions(CodeSource codesource) {
              return new PermissionsBuilder().addAll(getPermissions()).toPermissions();
--- 958,969 ----
              return permissions;
          }
  
          @Override
          public boolean implies(ProtectionDomain domain, Permission permission) {
!             return getPermissions().implies(permission) ||
!                     DEFAULT_POLICY.implies(domain, permission);
          }
  
          @Override
          public PermissionCollection getPermissions(CodeSource codesource) {
              return new PermissionsBuilder().addAll(getPermissions()).toPermissions();