zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mahadev konar (JIRA)" <j...@apache.org>
Subject [jira] Updated: (ZOOKEEPER-68) parseACLs in ZooKeeper.java fails to parse elements of ACL, should be lastIndexOf rather than IndexOf
Date Mon, 07 Jul 2008 22:40:31 GMT

     [ https://issues.apache.org/jira/browse/ZOOKEEPER-68?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Mahadev konar updated ZOOKEEPER-68:
-----------------------------------

    Attachment: ZOOKEEPER-68_1.patch

the patch seems fine but not generated from the trunk. you should do an svn diff from the
top level directory. I created a patch thats from the trunk and uploaded it.

> parseACLs in ZooKeeper.java fails to parse elements of ACL, should be lastIndexOf rather
than IndexOf
> -----------------------------------------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-68
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-68
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: java client
>            Reporter: Jakob Homan
>            Priority: Minor
>         Attachments: colonindices.diff, ZOOKEEPER-68_1.patch
>
>
> the parseACLs(String aclString) method attempts to prase ACLs from the form of scheme:id:perm
into the three components, delineated by the colons.  The current version calls indexOf for
both the first and second colon, receiving the same value for both and failing as if there
were only one colon in the string.  I created a one-line patch to call lastIndexOf for the
second colon.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message