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

Test for pending watches in send_set_watches should be moved

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 3.3.0
    • c client
    • None
    • 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
          Benjamin Reed
        2. ZOOKEEPER-622.patch
          22 kB
          Steven Cheng
        3. ZOOKEEPER-622.patch
          20 kB
          Steven Cheng
        4. ZOOKEEPER-622.patch
          1 kB
          Steven Cheng

        Issue Links

          Activity

            People

              breed Benjamin Reed
              stevechy Steven Cheng
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: