On Mon, 30 Sep 2024 16:11:27 GMT, SendaoYan <s...@openjdk.org> wrote:
> Hi all, > Test `com/sun/tools/attach/PermissionTest.java` fails access denied after > [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). This testcase > need the `readlink` permission of file `/proc/self/ns/mnt` after > [JDK-8327114](https://bugs.openjdk.org/browse/JDK-8327114). > So this PR add `readlink` permission to make this test work normally. > Before this PR, test run failed, after this PR, test run success. Test fix > only, no risk. test/jdk/com/sun/tools/attach/java.policy.allow line 17: > 15: > 16: /* to read configuration file in META-INF/services, and write/delete > .attach_pid<pid> */ > 17: /* to read symbol link in /proc/self/ns/mnt */ Suggestion: /* to read symbolic link of /proc/self/ns/mnt */ Perhaps merge the two comments enclosed with `/* ... */` on two lines? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21269#discussion_r1781479061
