From 3051df8961273fb39fe282a46c7f76124b39112f Mon Sep 17 00:00:00 2001 From: Norman Maurer Date: Wed, 31 Aug 2016 13:19:26 +0200 Subject: [PATCH] Ensure accessing System property is done via AccessController. Motivation: When a SecurityManager is in place it may dissallow accessing the property which will lead to not be able to load the application. Modifications: Use AccessController.doPrivileged(...) Result: No more problems with SecurityManager. --- .../netty/handler/ssl/ReferenceCountedOpenSslContext.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/handler/src/main/java/io/netty/handler/ssl/ReferenceCountedOpenSslContext.java b/handler/src/main/java/io/netty/handler/ssl/ReferenceCountedOpenSslContext.java index 0e3064857b..8d81343efa 100644 --- a/handler/src/main/java/io/netty/handler/ssl/ReferenceCountedOpenSslContext.java +++ b/handler/src/main/java/io/netty/handler/ssl/ReferenceCountedOpenSslContext.java @@ -79,7 +79,12 @@ public abstract class ReferenceCountedOpenSslContext extends SslContext implemen * Significant SSL/TLS improvements in Java 8 */ private static final boolean JDK_REJECT_CLIENT_INITIATED_RENEGOTIATION = - SystemPropertyUtil.getBoolean("jdk.tls.rejectClientInitiatedRenegotiation", false); + AccessController.doPrivileged(new PrivilegedAction() { + @Override + public Boolean run() { + return SystemPropertyUtil.getBoolean("jdk.tls.rejectClientInitiatedRenegotiation", false); + } + }); private static final List DEFAULT_CIPHERS; private static final Integer DH_KEY_LENGTH; private static final ResourceLeakDetector leakDetector =