From 814cdd2fae28920599ca801109c8adcf0f836d3c Mon Sep 17 00:00:00 2001
From: Richard Millet <richard.millet@berkeley.edu>
Date: Wed, 16 Jun 2010 04:34:33 +0000
Subject: [PATCH] CSPACE-2151: Delete permissions on specified SID is deleting
 incorrect permissions.

---
 .../services/authorization/spring/SpringPermissionManager.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/services/authorization/service/src/main/java/org/collectionspace/services/authorization/spring/SpringPermissionManager.java b/services/authorization/service/src/main/java/org/collectionspace/services/authorization/spring/SpringPermissionManager.java
index 6924e472e..a8ab08b9c 100644
--- a/services/authorization/service/src/main/java/org/collectionspace/services/authorization/spring/SpringPermissionManager.java
+++ b/services/authorization/service/src/main/java/org/collectionspace/services/authorization/spring/SpringPermissionManager.java
@@ -354,14 +354,13 @@ public class SpringPermissionManager implements CSpacePermissionManager {
                 if (ace.getSid().equals(sid)
                         && ace.getPermission().equals(permission)) {
                     foundAces.add(i);
-                    i++;
                 }
             } else {
                 if (ace.getPermission().equals(permission)) {
                     foundAces.add(i);
-                    i++;
                 }
             }
+            i++;
         }
         for (int j = foundAces.size() - 1; j >= 0; j--) {
             //the following operation does not work while iterating in the while loop
-- 
2.47.3