On Wed, 27 Nov 2024 12:24:54 GMT, Aleksei Efimov <aefi...@openjdk.org> wrote:

>> This PR removes calls to and mentions of the `SecurityManager` and the 
>> `doPrivileged` from the `java.net.InetAddress` and 
>> `sun.net.util.IPAddressUtil` and related classes. 
>> 
>> Noteworthy changes:
>> - the default value (`30s`) of the `networkaddress.cache.ttl` security 
>> property responsible for successfull address lookups  kept unchanged, the 
>> mention of SM removed from the property description in the 
>> `net-properties.html` file.
>> - connect permission checks are removed from the `InetAddress`, and relevant 
>> classes, like `SocketPermission`. `SocketPermission.initEphemeralPorts` was 
>> also modified to remove `doPrivileged`.
>> - `RuntimePermission("inetAddressResolverProvider")` permission has been 
>> removed from the system-wide `InetAddressResolver` initialization code.
>> 
>> `tier1` to `tier3` tests show no relevant failures.
>
> Aleksei Efimov has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Replace missed GetPropertyAction.privilegedGetProperty usages

LGTM

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

Marked as reviewed by dfuchs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22376#pullrequestreview-2465093763

Reply via email to