You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ad...@apache.org on 2004/11/12 17:13:07 UTC

svn commit: rev 57530 - geronimo/trunk/modules/security/src/java/org/apache/geronimo/security

Author: adc
Date: Fri Nov 12 08:13:07 2004
New Revision: 57530

Modified:
   geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/SecurityService.java
Log:
A GBean's constructor must not call its member functions.

Modified: geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/SecurityService.java
==============================================================================
--- geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/SecurityService.java	(original)
+++ geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/SecurityService.java	Fri Nov 12 08:13:07 2004
@@ -82,7 +82,12 @@
 
             throw (IllegalStateException) new IllegalStateException().initCause(pce);
         }
-        setPolicyConfigurationFactory(policyConfigurationFactory);
+        
+        this.policyConfigurationFactory = policyConfigurationFactory;
+        //TODO remove this if wrapper when GeronimoMBean leaves.
+        if (policyConfigurationFactory != null) {
+            System.setProperty("javax.security.jacc.PolicyConfigurationFactory.provider", policyConfigurationFactory);
+        }
     }
 
     public String getPolicyConfigurationFactory() {