Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-847

Missing acl check in zookeeper create

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 3.3.1, 3.3.2, 3.3.3
    • None
    • java client
    • None
    • acl-check

    Description

      I watched the source of the zookeeper class and I missed an acl check in the asynchronous version of the create operation. Is there any reason, that in the asynch version is no
      check whether the acl is valid, or did someone forget to implement it. It's interesting because we worked on a refactoring of the zookeeper client and don't want to implement a bug.

      The following code is missing:
      if (acl != null && acl.size() == 0)

      { throw new KeeperException.InvalidACLException(); }

      Attachments

        1. ZOOKEEPER-847.patch
          12 kB
          Laxman

        Activity

          People

            Unassigned Unassigned
            ymc-pada Patrick Datko
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated: