Please review this PR which cleans up SecurityManager related code from 
`java.net.NetworkInterface` after JEP-486.

The specification of four methods in NetworkInterface is updated to reflect 
that these methods no longer uses `SecurityManager` to filter the results such 
that a subset may be returned. A CSR draft has been filed 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 results pending.

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

Commit messages:
 - Remove usages of SecurityManager
 - Update the spec of getHardwareAddress to reflect that this method no longer 
filters for privileges using SecurityManager
 - Leftover API spec updated for getInetAddresses, inetAddresses and 
getInterfaceAddresses

Changes: https://git.openjdk.org/jdk/pull/22177/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22177&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8344218
  Stats: 78 lines in 1 file changed: 3 ins; 55 del; 20 mod
  Patch: https://git.openjdk.org/jdk/pull/22177.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22177/head:pull/22177

PR: https://git.openjdk.org/jdk/pull/22177

Reply via email to