Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0.0 M1
    • Component/s: ACLs
    • Labels:
      None

      Description

      On line 130 of AclImpl.java in trunk (line number is different in other releases) a check is performed on the upper bound of the list of access control entries.

      Code is:
      if (aceIndex > this.aces.size()) {

      should be:
      if (aceIndex >= this.aces.size()) {

      Result is that exception out of underlying list impl is thrown. Minor.

        Issue Links

          Activity

          Hide
          Luke Taylor added a comment -

          Thanks for the report. I've updated AclImpl and added some extra tests.

          I'm not sure that an IndexOutOfBoundsException might not be more appopriate in this case but we'll stick with the existing API.

          Show
          Luke Taylor added a comment - Thanks for the report. I've updated AclImpl and added some extra tests. I'm not sure that an IndexOutOfBoundsException might not be more appopriate in this case but we'll stick with the existing API.

            People

            • Assignee:
              Luke Taylor
              Reporter:
              Taylor Mathewson
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: