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

Test for pending watches in send_set_watches should be moved

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3.0
    • Component/s: c client
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Valgrind found:

      ==2357== Conditional jump or move depends on uninitialised value(s)
      ==2357== at 0x807FDCA: check_events (zookeeper.c:1180)
      ==2357== by 0x808043A: zookeeper_process (zookeeper.c:1775)
      ==2357== by 0x806A21B: Zookeeper_close::testCloseConnected1() (TestZookeeperClose.cc:161)
      ==2357== by 0x806C6BF: CppUnit::TestCaller<Zookeeper_close>::runTest() (TestCaller.h:166)

      zookeeper.c:1180 was the first if in send_set_watches.

        Attachments

        1. ZOOKEEPER-622.patch
          1 kB
          Steven Cheng
        2. ZOOKEEPER-622.patch
          20 kB
          Steven Cheng
        3. ZOOKEEPER-622.patch
          22 kB
          Steven Cheng
        4. ZOOKEEPER-622.patch
          1 kB
          Benjamin Reed

          Issue Links

            Activity

              People

              • Assignee:
                breed Benjamin Reed
                Reporter:
                stevechy Steven Cheng
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: