On Sun, 17 Nov 2024 09:28:47 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.
>
> Eirik Bjørsnøs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert method description changes for getHardwareAddress

Good to see this code go away.

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

PR Review: https://git.openjdk.org/jdk/pull/22177#pullrequestreview-2442743947

Reply via email to