client: check wait return code in waitAny

Currently, we do not check the return code of `wait` in `waitAny`. If there
is an error, we must log it and return a `nullopt`.

Closes #121

Author: CuriousGeorgiy

src/Client/Connector.hpp

@@ -301,7 +301,10 @@ Connector<BUFFER, NetProvider>::waitAny(int timeout)
 	Timer timer{timeout};
 	timer.start();
 	while (m_ReadyToDecode.empty()) {
-		m_NetProvider.wait(timer.timeLeft());
+		if (m_NetProvider.wait(timer.timeLeft()) != 0) {
+			LOG_ERROR("Failed to poll connections: ", strerror(errno));
+			return std::nullopt;
+		}
 		if (timer.isExpired())
 			break;
 	}

test/ClientTest.cpp

@@ -32,8 +32,10 @@
 #include "Utils/System.hpp"
 #include "Utils/UserTuple.hpp"
 
-#include "../src/Client/LibevNetProvider.hpp"
 #include "../src/Client/Connector.hpp"
+#include "../src/Client/LibevNetProvider.hpp"
+
+#include <thread>
 
 const char *localhost = "127.0.0.1";
 int port = 3301;
@@ -1144,6 +1146,11 @@ response_decoding(Connector<BUFFER, NetProvider> &client)
 	client.close(conn);
 }
 
+void sigusr_handler(int signo)
+{
+	fail_unless(signo != SIGINT);
+}
+
 /** Checks all available `wait` methods of connector. */
 template <class BUFFER, class NetProvider>
 void
@@ -1324,6 +1331,30 @@ test_wait(Connector<BUFFER, NetProvider> &client)
 	fail_unless(result.header.sync == static_cast<int>(f1));
 	fail_unless(result.header.code == 0);
 
+#ifdef __linux__
+	TEST_CASE("waitAny internal wait failure (gh-121)");
+	static constexpr double LONG_SLEEP_TIME = 2;
+	f = conn.call("remote_sleep", std::forward_as_tuple(LONG_SLEEP_TIME));
+	fail_unless(!conn.futureIsReady(f));
+	pthread_t tid = pthread_self();
+	struct sigaction act;
+	act.sa_handler = sigusr_handler;
+	act.sa_flags = 0;
+	sigemptyset(&act.sa_mask);
+	sigaction(SIGUSR1, &act, nullptr);
+	auto &&signal_thread = std::thread([tid] {
+		sleep(LONG_SLEEP_TIME / 2);
+		pthread_kill(tid, SIGUSR1);
+	});
+	fail_unless(!client.waitAny().has_value());
+	fail_unless(client.waitAny().has_value());
+	fail_unless(conn.futureIsReady(f));
+	response = conn.getResponse(f);
+	fail_unless(response.has_value());
+	signal_thread.join();
+	sigaction(SIGUSR1, nullptr, nullptr);
+#endif /* __linux__ */
+
 	client.close(conn);
 }