From: Richard Millet <richard.millet@berkeley.edu>
Date: Wed, 16 Jun 2010 04:34:33 +0000 (+0000)
Subject: CSPACE-2151: Delete permissions on specified SID is deleting incorrect permissions.
X-Git-Url: https://git.aero2k.de/?a=commitdiff_plain;h=814cdd2fae28920599ca801109c8adcf0f836d3c;p=tmp%2Fjakarta-migration.git

CSPACE-2151: Delete permissions on specified SID is deleting incorrect permissions.
---

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