RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

Alan Bateman alanb at openjdk.org
Thu Nov 14 17:17:11 UTC 2024


On Thu, 14 Nov 2024 12:57:25 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

> Please review this PR which removes uses of `AccessController::doPrivileged` in `sun/net/ftp`.
> 
> Changes are mostly straightforward, one exception is the removal of the somewhat exotic unwrapping of temporary arrays in the static initializer of `impl/FtpClient`.
> 
> Testing: `test/jdk/sun/net/ftp` run green. GHA results pending.

Looks okay to me, didn't spot anything wrong or missing.

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

Marked as reviewed by alanb (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22105#pullrequestreview-2436731487


More information about the net-dev mailing list