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

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

Author: david.lloyd at jboss.com
Date: 2009-06-22 12:48:43 -0400 (Mon, 22 Jun 2009)
New Revision: 3300

Modified:
   jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java
Log:
Creating a logger is privileged, so encompass it in a priv block (JBLOGGING-18 part 2)

Modified: jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java
===================================================================
--- jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java	2009-06-22 15:43:38 UTC (rev 3299)
+++ jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java	2009-06-22 16:48:43 UTC (rev 3300)
@@ -166,8 +166,6 @@
             }
             final Logger logger = createLogger(fullName);
             if (loggerRefUpdater.compareAndSet(this, loggerRef, parent == null ? new StrongLoggerRef(logger) : new WeakLoggerRef(logger))) {
-                // initialize the effective level
-                logger.setLevel(null);
                 return logger;
             }
         }
@@ -178,11 +176,17 @@
         if (sm != null) {
             return AccessController.doPrivileged(new PrivilegedAction<Logger>() {
                 public Logger run() {
-                    return new Logger(LoggerNode.this, fullName);
+                    final Logger logger = new Logger(LoggerNode.this, fullName);
+                    // initialize the effective level
+                    logger.setLevel(null);
+                    return logger;
                 }
             });
         } else {
-            return new Logger(this, fullName);
+            final Logger logger = new Logger(this, fullName);
+            // initialize the effective level
+            logger.setLevel(null);
+            return logger;
         }
     }