RFR: 8341246: Test com/sun/tools/attach/PermissionTest.java fails access denied after JDK-8327114 [v2]
SendaoYan
syan at openjdk.org
Tue Oct 1 15:38:56 UTC 2024
On Mon, 30 Sep 2024 17:05:08 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
>> SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> merge two comments enclosed with /* ... */ on two lines
>
> 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?
Okey, the two comments has been merged.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21269#discussion_r1783093102
More information about the serviceability-dev
mailing list