- 24 Apr, 2016 1 commit
-
-
hitstergtd authored
Solution: - Adjust test_xpub_proxy_unsubscribe_disconnect() to support different protocol types - Exclude the IPC tests on Windows and OpenVMS H/T: @somdoron
-
- 23 Apr, 2016 4 commits
-
-
Doron Somech authored
-
Doron Somech authored
-
Luca Boccassi authored
Solution: check that both pointers passed as arguments to test_missing_subscription are non-NULL, instead of the opposite.
-
hitstergtd authored
Solution: - Adjust test_subscriptions() to support different protocol types - Run TCP and IPC tests everywhere but on Windows and OpenVMS
-
- 21 Apr, 2016 5 commits
-
-
hitstergtd authored
Solution: PR #1906 did not solve this problem properly; subsequent Travis CI indicated that the issue happens with Clang/LLVM, so make sure to fix the issue by detecting if Cmake CMake is using Clang for building the tests.
-
Luca Boccassi authored
Solution: move it to the unix-only section of Makefile.am and tests/CMakeLists.txt since it uses Unix IPC sockets.
-
hitstergtd authored
Solution: add Winsock specific assertions, since getpeername() should will return SOCKET_ERROR (-1) and WSAGetLastError() will be set to WSAENOTSOCK
-
somdoron authored
-
Luca Boccassi authored
Solution: add macro in ZMQSourceRunChecks.cmake and optionally include the TIPC sources if the support is available. More importantly, only run the TIPC tests if the support is there.
-
- 19 Apr, 2016 1 commit
-
-
hitstergtd authored
Problem: A per-test Linker search path was added in commit a911fa41 to CMakeLists.txt as part of fixing Windows builds. Whilst this is silently ignored by ld(1) on Linux, it doesn't settle well with OS X. Spurious warnings are generated about missing directories leading to convoluted build logs. Solution: Make per-Test LINK_DIRECTORIES() conditional for non-Apple platforms.
-
- 18 Apr, 2016 4 commits
-
-
Luca Boccassi authored
Solution: try to resolve the TCP endpoint passed by the user in the zmq_unbind call before giving up, if it doesn't match. This fixes a breakage in the API, where after a call to zmq_bind(s, "tcp://127.0.0.1:9999") with IPv6 enabled on s would result in the call to zmq_unbind(s, "tcp://127.0.0.1:9999") failing. Add more test cases to increase coverage on all combinations of TCP endpoints.
-
Luca Boccassi authored
Solution: add helper function is_ipv6_available to testutil.hpp to test if IPv6 is available on the building platform. This function will try to open and bind a socket to ::1:*, as it's the ultimate way of knowing if, at least on the loopback, IPv6 is enabled.
-
hitstergtd authored
Solution: fix comment
-
hitstergtd authored
Solution: - call zmq_msg_close() if there is an error - call free() to release resources if there is an error (CWE-404)
-
- 14 Apr, 2016 1 commit
-
-
hitstergtd authored
Move tests specific to Linux under a platform conditional thereby eliminating unnecessary builds and fixing "make test" on Mac OS X and possibly other non-Linux systems. Tests specific to Linux: - abstract namespace support for AF_UNIX sockets - TIPC support (AF_TIPC) Test success rate jumps from 90% to 100% on Mac OS X after this change.
-
- 12 Apr, 2016 2 commits
-
-
Luca Boccassi authored
Solution: pass a struct addrinfo hint to getaddrinfo with a hint about the address family to avoid a failure.
-
Patrik Wenger authored
Solution: return -1 (no event) instead of 0 (event) For some reason, this just returns 0 if there are no sockets registered on the poller. Usually this would mean there has been an event. So the caller would have to check the return value AND the event, or write code that takes the number of registered sockets into consideration. By returning -1 and setting errno = ETIMEDOUT like in the usual timeout cases, it's more consistent and convenient. Test case included.
-
- 08 Apr, 2016 1 commit
-
-
Satyajit Padalkar authored
-
- 02 Apr, 2016 1 commit
-
-
Frederic Tregon authored
Problem: Since pull request #1730 was merged, protocol for REQ socket is checked at the session level and this check does not take into account the possibility of a request_id being part of the message. Thus the option ZMQ_REQ_CORRELATE would no longer work. This is now fixed: the possiblity of a 4 bytes integer being present before the delimiter frame is taken into account (whether or not this breaks the REQ/REP RFC is another issue).
-
- 20 Mar, 2016 1 commit
-
-
Frederic Tregon authored
Problem: when using ZMQ_REQ_RELAXED + ZMQ_REQ_CORRELATE and two 'send' are executed in a row and no server is available at the time of the sends, then the internal request_id used to identify messages gets corrupted and the two messages end up with the same request_id. The correlation no longer works in that case and you may end up with the wrong message. Solution: make a copy of the request_id instance member before sending it down the pipe.
-
- 17 Mar, 2016 1 commit
-
-
Pete LaDow authored
-
- 16 Mar, 2016 1 commit
-
-
Luca Boccassi authored
Solution: remove temporarily until proper message limits have been implemented, then a more granular test case can be added without such high requirements which are problematic in embedded environment, build systems, VMs and CI systems
-
- 14 Mar, 2016 2 commits
-
-
Luca Boccassi authored
Solution: increase path buffer length to 73
-
Suudy authored
Updated handling of Unix Domain Sockets, make use of temporary directories, and cleanup afterward. Fix test_term_endpoint handling of optvallen
-
- 06 Mar, 2016 1 commit
-
-
Osiris authored
Solution: Add precompiled flags to CMakeList.txt for faster compiles + bonus - removed compilation warning on Windows by adding add_definitions (-D_WINSOCK_DEPRECATED_NO_WARNINGS)
-
- 05 Mar, 2016 2 commits
- 01 Mar, 2016 1 commit
-
-
Pieter Hintjens authored
Libsodium has started returning -1 in some cases. Solution: allow and handle error returns from these calls. Fixes #1831
-
- 13 Feb, 2016 11 commits
-
-
Luca Boccassi authored
Solution: increase SETTLE_TIME from 50ms to 300ms to avoid failing on slower environments like through Valgrind on slow VMs.
-
Luca Boccassi authored
Solution: suggest to use the common approach of msleep (SETTLE_TIME) after a connect if necessary, rather than reimplementing a different way.
-
Luca Boccassi authored
Solution: include pthread.h in testutil.hpp, removed from test_shutdown_stress_tipc in an earlier commit
-
Luca Boccassi authored
Solution: add msleep (SETTLE_TIME) to test_immediate, test_spec_rep and test_spec_router after the sockets are created and connected to avoid failing when running in slower environment like through Valgrind in underpowered VMs.
-
Luca Boccassi authored
Solution: use msleep (SETTLE_TIME) everywhere when waiting for the connections/sockets to be settled instead of a variety of patterns and functions to make tests more coherent.
-
Luca Boccassi authored
Solution: memset temporary buffer before use to reduce Valgrind noise
-
Luca Boccassi authored
Solution: memset temporary buffer before use to reduce Valgrind noise
-
Luca Boccassi authored
Solution: close socket and destroy context to reduce Valgrind noise
-
Luca Boccassi authored
Solution: memset temporary buffer before use to reduce Valgrind noise
-
Luca Boccassi authored
Solution: free addrinfo to reduce Valgrind noise
-
Luca Boccassi authored
Solution: close socket and destroy context to reduce Valgrind noise
-