geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xuhaih...@apache.org
Subject svn commit: r781640 - /geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbDeploymentBuilder.java
Date Thu, 04 Jun 2009 02:21:26 GMT
Author: xuhaihong
Date: Thu Jun  4 02:21:26 2009
New Revision: 781640

URL: http://svn.apache.org/viewvc?rev=781640&view=rev
Log:
GERONIMO-4669 EJB security does not work correctly when no permssion is set and the user does
a login 

Modified:
    geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbDeploymentBuilder.java

Modified: geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbDeploymentBuilder.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbDeploymentBuilder.java?rev=781640&r1=781639&r2=781640&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbDeploymentBuilder.java
(original)
+++ geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbDeploymentBuilder.java
Thu Jun  4 02:21:26 2009
@@ -178,7 +178,7 @@
 
     public ComponentPermissions buildComponentPermissions() throws DeploymentException {
         List<MethodPermission> methodPermissions = ejbModule.getEjbJar().getAssemblyDescriptor().getMethodPermission();
-        if (!methodPermissions.isEmpty()) {
+        if (earContext.getSecurityConfiguration() != null) {
             earContext.setHasSecurity(true);
         }
         if (earContext.getSecurityConfiguration() == null && methodPermissions.size()
> 0) {



Mime
View raw message