Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0, 3.0.1, 3.1.0
    • Fix Version/s: 3.1.1, 3.2.0
    • Component/s: c client
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      The zookeeper_process() function incorrectly calls the c.acl_result member of the completion_list_t structure when handling the completion from a synchronous zoo_get_acl() request. The c.acl_result member is set to SYNCHRONOUS_MARKER, which is a null pointer.

      The attached patch removes this call.

      1. ZOOKEEPER-309.patch
        0.6 kB
        Chris Darroch
      2. ZOOKEEPER-309.patch
        4 kB
        Mahadev konar

        Activity

        Hide
        Benjamin Reed added a comment -

        +1 that's embarrassing!

        Show
        Benjamin Reed added a comment - +1 that's embarrassing!
        Hide
        Mahadev konar added a comment -

        can we include a test case for this so that we do not get embarrased again?

        Show
        Mahadev konar added a comment - can we include a test case for this so that we do not get embarrased again?
        Hide
        Patrick Hunt added a comment -

        Moving from 3.1 to 3.1.1 – this is not a regression and has a workaround (too late in release cycle to address non-blocker for 3.1.0

        Show
        Patrick Hunt added a comment - Moving from 3.1 to 3.1.1 – this is not a regression and has a workaround (too late in release cycle to address non-blocker for 3.1.0
        Hide
        Patrick Hunt added a comment -

        Cancelling patch - needs tests. I suggest that tests be added for ACLs in general (not just this api) – the cppunits are missing any kind of acl testing.

        Show
        Patrick Hunt added a comment - Cancelling patch - needs tests. I suggest that tests be added for ACLs in general (not just this api) – the cppunits are missing any kind of acl testing.
        Hide
        Chris Darroch added a comment -

        Renamed patch file with issue key.

        Show
        Chris Darroch added a comment - Renamed patch file with issue key.
        Hide
        Mahadev konar added a comment -

        this patch adds some tests to chris's patch. The test runs the sync setacl's and getacl's and since both of them use the async version of the set/get acl, it indirectly tests the async versions as well.

        Show
        Mahadev konar added a comment - this patch adds some tests to chris's patch. The test runs the sync setacl's and getacl's and since both of them use the async version of the set/get acl, it indirectly tests the async versions as well.
        Hide
        Mahadev konar added a comment -

        also forgot to mention that the tests fail with segmenetation fault without chris's patch and pass with the patch..

        Show
        Mahadev konar added a comment - also forgot to mention that the tests fail with segmenetation fault without chris's patch and pass with the patch..
        Hide
        Patrick Hunt added a comment -

        +1, good to see we are testing acls now in c client.

        Committed revision 752729.
        Committed revision 752730.

        Show
        Patrick Hunt added a comment - +1, good to see we are testing acls now in c client. Committed revision 752729. Committed revision 752730.
        Hide
        Hudson added a comment -

        Integrated in ZooKeeper-trunk #253 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/253/)

        Show
        Hudson added a comment - Integrated in ZooKeeper-trunk #253 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/253/ )

          People

          • Assignee:
            Mahadev konar
            Reporter:
            Chris Darroch
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development