On Sat, 16 Nov 2024 21:01:56 GMT, Eirik Bjørsnøs <eir...@openjdk.org> wrote:
> Please review this PR which cleans up SecurityManager related code from > `java.net.NetworkInterface` after JEP-486. > > The specification of three methods in NetworkInterface is updated to reflect > that these methods no longer use `SecurityManager` to filter their results > such that a subset may be returned. A CSR draft has been proposed to fix this > leftover from JEP-486. > > The actual SecurityManager cleanup: > > * The `getCheckedInetAddresses` method is removed. > * `getInetAddresses` is updated to return `enumerationFromArray(addrs)` (with > no filtering) > * `inetAddresses` is update to return `streamFromArray(addrs)` (with no > filtering) > * `getInterfaceAddresses` is updated to return the list of InterfaceAddresses > (with no filtering) > * `getHardwareAddress` is updated to return the result with no permission > checking > * Three c-style array declarations hurt my eyes and were updated (sorry!) > > GHA and tier2 results pending. This pull request has now been integrated. Changeset: b12c5b4d Author: Eirik Bjørsnøs <eir...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/b12c5b4d18d9bd53e44e515ac1fac548ceeb3dc6 Stats: 76 lines in 1 file changed: 3 ins; 55 del; 18 mod 8344218: Remove calls to SecurityManager and and doPrivileged in java.net.NetworkInterface after JEP 486 integration Reviewed-by: dfuchs ------------- PR: https://git.openjdk.org/jdk/pull/22177