From 797fe12f3a1bb982fb5a1047cee30cfd1a7bef81 Mon Sep 17 00:00:00 2001
From: David Ostrovsky <david@ostrovsky.org>
Date: Sun, 30 Oct 2016 11:00:38 +0100
Subject: [PATCH] [SSHD-706] AbstractSession#checkForTimeouts: Avoid eager
 log4j invocations

Without Logger.isDebugEnabled() guard, the log message was always
constructed even though the log level was less than debug.
---
 .../apache/sshd/common/session/helpers/AbstractSession.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/sshd-core/src/main/java/org/apache/sshd/common/session/helpers/AbstractSession.java b/sshd-core/src/main/java/org/apache/sshd/common/session/helpers/AbstractSession.java
index 72acbc381..89263eb5a 100644
--- a/sshd-core/src/main/java/org/apache/sshd/common/session/helpers/AbstractSession.java
+++ b/sshd-core/src/main/java/org/apache/sshd/common/session/helpers/AbstractSession.java
@@ -2365,8 +2365,10 @@ protected Map<KexProposalOption, String> mergeProposals(Map<KexProposalOption, S
      */
     protected void checkForTimeouts() throws IOException {
         if (isClosing()) {
-            log.debug("checkForTimeouts({}) session closing", this);
-            return;
+            if (log.isDebugEnabled()) {
+                log.debug("checkForTimeouts({}) session closing", this);
+                return;
+            }
         }
 
         long now = System.currentTimeMillis();