[ZOOKEEPER-622] Test for pending watches in send_set_watches should be moved

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.3.0
• Component/s: c client
• Hadoop Flags: Reviewed
• Assignee: Benjamin Reed
• Reporter: Steven Cheng
• Created: 12/Dec/09 22:22
• Updated: 26/Mar/10 17:25
• Resolved: 06/Mar/10 05:35

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.

Issue Links

• is related to: ZOOKEEPER-685 make C system tests use multiple servers