[jboss-svn-commits] JBoss Common SVN: r4886 - slf4j-jboss-logmanager/trunk/src/main/java/org/slf4j/impl.

[jboss-svn-commits at lists.jboss.org]

Author: david.lloyd at jboss.com
Date: 2010-12-23 10:56:29 -0500 (Thu, 23 Dec 2010)
New Revision: 4886

Modified:
   slf4j-jboss-logmanager/trunk/src/main/java/org/slf4j/impl/Slf4jLogger.java
Log:
[JBAS-8772] Prevent bypass of level check

Modified: slf4j-jboss-logmanager/trunk/src/main/java/org/slf4j/impl/Slf4jLogger.java
===================================================================
--- slf4j-jboss-logmanager/trunk/src/main/java/org/slf4j/impl/Slf4jLogger.java	2010-12-16 14:55:07 UTC (rev 4885)
+++ slf4j-jboss-logmanager/trunk/src/main/java/org/slf4j/impl/Slf4jLogger.java	2010-12-23 15:56:29 UTC (rev 4886)
@@ -64,9 +64,11 @@
             case LocationAwareLogger.ERROR_INT: level = org.jboss.logmanager.Level.ERROR; break;
             default: level = org.jboss.logmanager.Level.DEBUG; break;
         }
-        final ExtLogRecord rec = new ExtLogRecord(level, message, fqcn);
-        rec.setThrown(t);
-        logger.logRaw(rec);
+        if (logger.isLoggable(level)) {
+            final ExtLogRecord rec = new ExtLogRecord(level, message, fqcn);
+            rec.setThrown(t);
+            logger.logRaw(rec);
+        }
     }
 
     /** {@inheritDoc} */