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

Eirik Bjørsnøs eirbjo at openjdk.org
Thu Nov 14 20:18:27 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.

I ran `jtreg:test/jdk:tier2` locally.

3 of 4114 tests failed.

Two are TLS tests failing with `SSLHandshakeException: Remote host terminated the handshake`. Seems intermittent.

`VerifyJimage` fails with 'IncompatibleClassChangeError: class java.util.concurrent.StructuredTaskScopeImpl$SubtaskImpl cannot implement sealed interface java.util.concurrent.StructuredTaskScope$Subtask'. Seems unrelated.

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

PR Comment: https://git.openjdk.org/jdk/pull/22105#issuecomment-2477325844


More information about the net-dev mailing list