[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Integrated: 8267184: Add -Djava.security.manager=allow to tests calling System.setSecurityManager
From:       Weijun Wang <weijun () openjdk ! java ! net>
Date:       2021-05-24 17:02:13
Message-ID: 3rH0Qzq38uj4gzgkoDyLE_SnE47c8710ZvGAeSK6UA4=.e080d8bc-838b-4d62-87ff-ca8b12445c8a () github ! com
[Download RAW message or body]

On Mon, 17 May 2021 17:51:36 GMT, Weijun Wang <weijun@openjdk.org> wrote:

> Please review the test changes for [JEP 411](https://openjdk.java.net/jeps/411).
> 
> With JEP 411 and the default value of `-Djava.security.manager` becoming \
> `disallow`, tests calling `System.setSecurityManager()`  need \
> `-Djava.security.manager=allow` when launched. This PR covers such changes for \
> tier1 to tier3 (except for the JCK tests). 
> To make it easier to focus your review on the tests in your area, this PR is \
> divided into multiple commits for different areas (~~serviceability~~, \
> ~~hotspot-compiler~~, ~~i18n~~, ~~rmi~~, ~~javadoc~~, swing, 2d, ~~security~~, \
> ~~hotspot-runtime~~, ~~nio~~, ~~xml~~, ~~beans~~, ~~core-libs~~, ~~net~~, \
> ~~compiler~~, ~~hotspot-gc~~). Mostly the rule is the same as how Skara adds \
> labels, but there are several small tweaks: 
> 1. When a file is covered by multiple labels, only one is chosen. I make my best to \
> avoid putting too many tests into `core-libs`. If a file is covered by `core-libs` \
> and another label, I categorized it into the other label. 2. If a file is in \
> `core-libs` but contains `/xml/` in its name, it's in the `xml` commit. 3. If a \
> file is in `core-libs` but contains `/rmi/` in its name, it's in the `rmi` commit. \
> 4. One file not covered by any label -- \
> `test/jdk/com/sun/java/accessibility/util/8051626/Bug8051626.java` -- is in the \
> `swing` commit. 
> Due to the size of this PR, no attempt is made to update copyright years for all \
> files to minimize unnecessary merge conflict. 
> Please note that this PR can be integrated before the source changes for JEP 411, \
> as the possible values of this system property was already defined long time ago in \
> JDK 9. 
> Most of the change in this PR is a simple adding of `-Djava.security.manager=allow` \
> to the `@run main/othervm` line. Sometimes it was not `othervm` and we add one. \
> Sometimes there's no `@run` at all and we add the line. 
> There are several tests that launch another Java process that needs to call the \
> `System.setSecurityManager()` method, and the system property is added to \
> `ProcessBuilder`, `ProcessTools`, or the java command line (if the test is a shell \
> test). 
> 3 langtools tests are added into problem list due to \
> [JDK-8265611](https://bugs.openjdk.java.net/browse/JDK-8265611). 
> 2 SQL tests are moved because they need different options on the `@run` line but \
> they are inside a directory that has a `TEST.properties`: 
> rename test/jdk/java/sql/{testng/test/sql/othervm => \
> permissionTests}/DriverManagerPermissionsTests.java (93%) rename \
> test/jdk/javax/sql/{testng/test/rowset/spi => \
> permissionTests}/SyncFactoryPermissionsTests.java (95%) ```
> 
> The source change for JEP 411 is at https://github.com/openjdk/jdk/pull/4073.

This pull request has now been integrated.

Changeset: 640a2afd
Author:    Weijun Wang <weijun@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/640a2afda36857410b7abf398af81e35430a62e7
                
Stats:     1028 lines in 852 files changed: 252 ins; 9 del; 767 mod

8267184: Add -Djava.security.manager=allow to tests calling System.setSecurityManager

Co-authored-by: Lance Andersen <lancea@openjdk.org>
Co-authored-by: Weijun Wang <weijun@openjdk.org>
Reviewed-by: dholmes, alanb, dfuchs, mchung, mullan, prr

-------------

PR: https://git.openjdk.java.net/jdk/pull/4071


[prev in list] [next in list] [prev in thread] [next in thread]