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

Reply via email to