[jboss-cvs] jbosssx/src/main/org/jboss/security/authorization ...

[Anil Saldhana]

  User: asaldhana
  Date: 06/07/17 13:14:23

  Modified:    src/main/org/jboss/security/authorization 
                        AuthorizationContext.java
  Log:
  JBAS-3324: handle the abort and commit method replies from modules
  
  Revision  Changes    Path
  1.4       +10 -9     jbosssx/src/main/org/jboss/security/authorization/AuthorizationContext.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: AuthorizationContext.java
  ===================================================================
  RCS file: /cvsroot/jboss/jbosssx/src/main/org/jboss/security/authorization/AuthorizationContext.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -b -r1.3 -r1.4
  --- AuthorizationContext.java	29 Jun 2006 20:59:28 -0000	1.3
  +++ AuthorizationContext.java	17 Jul 2006 17:14:23 -0000	1.4
  @@ -21,7 +21,6 @@
    */
   package org.jboss.security.authorization;
   
  -import java.io.IOException;
   import java.security.AccessController;
   import java.security.PrivilegedActionException;
   import java.security.PrivilegedExceptionAction;
  @@ -43,7 +42,7 @@
   import org.jboss.security.config.ApplicationPolicy;
   import org.jboss.security.config.AuthorizationInfo;
   
  -//$Id: AuthorizationContext.java,v 1.3 2006/06/29 20:59:28 asaldhana Exp $
  +//$Id: AuthorizationContext.java,v 1.4 2006/07/17 17:14:23 asaldhana Exp $
   
   /**
    *  Authorization Framework for Policy Decision Modules
  @@ -59,7 +58,7 @@
    *  
    *  @author <a href="mailto:Anil.Saldhana at jboss.org">Anil Saldhana</a>
    *  @since  Jun 11, 2006 
  - *  @version $Revision: 1.3 $
  + *  @version $Revision: 1.4 $
    */
   public class AuthorizationContext
   {
  @@ -271,8 +270,9 @@
         for(int i = 0; i < length; i++)
         {
            AuthorizationModule module = (AuthorizationModule)modules.get(i);
  -         //TODO: what to do with bool?
            boolean bool = module.commit(); 
  +         if(!bool)
  +            throw new AuthorizationException("commit on modules failed");
         } 
      }
      
  @@ -283,8 +283,9 @@
         for(int i = 0; i < length; i++)
         {
            AuthorizationModule module = (AuthorizationModule)modules.get(i);
  -         //TODO: what to do with bool?
            boolean bool = module.abort(); 
  +         if(!bool)
  +            throw new AuthorizationException("abort on modules failed");
         } 
      }