ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-1380

zkperl: _zk_release_watch doesn't remove items properly from the watch list

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.3.3, 3.3.4, 3.4.0, 3.4.1, 3.4.2
    • Fix Version/s: 3.4.4, 3.5.0
    • Component/s: contrib-bindings
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Tags:
      zookeeper perl zkperl Net-ZooKeeper

      Description

      The doubly linked list of watches is not updated properly if a watch is taken out from the middle of the chain.
      The item after the item which is taken out will receive null pointer for the previous element! This will make the doubly linked list inconsistent and can lead to segfault or infinite loop when the doubly linked list is iterated later.

        Activity

        Hide
        Botond Hejj added a comment -

        Patch to fix the doubly linked list update problem

        Show
        Botond Hejj added a comment - Patch to fix the doubly linked list update problem
        Hide
        Mahadev konar added a comment -

        Looks good. Committed this to trunk and 3.4 branch.

        Show
        Mahadev konar added a comment - Looks good. Committed this to trunk and 3.4 branch.
        Hide
        Hudson added a comment -

        Integrated in ZooKeeper-trunk #1672 (See https://builds.apache.org/job/ZooKeeper-trunk/1672/)
        ZOOKEEPER-1380. zkperl: _zk_release_watch doesn't remove items properly from the watch list. (Botond Hejj via mahadev) (Revision 1381897)

        Result = SUCCESS
        mahadev : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1381897
        Files :

        • /zookeeper/trunk/CHANGES.txt
        • /zookeeper/trunk/src/contrib/zkperl/ZooKeeper.xs
        Show
        Hudson added a comment - Integrated in ZooKeeper-trunk #1672 (See https://builds.apache.org/job/ZooKeeper-trunk/1672/ ) ZOOKEEPER-1380 . zkperl: _zk_release_watch doesn't remove items properly from the watch list. (Botond Hejj via mahadev) (Revision 1381897) Result = SUCCESS mahadev : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1381897 Files : /zookeeper/trunk/CHANGES.txt /zookeeper/trunk/src/contrib/zkperl/ZooKeeper.xs

          People

          • Assignee:
            Botond Hejj
            Reporter:
            Botond Hejj
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development