You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xu...@apache.org on 2009/06/04 04:21:26 UTC
svn commit: r781640 -
/geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbDeploymentBuilder.java
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) {