zeroconf/Bonjour: fix OnSocketReady() return value

Keep the SocketMonitor registered. This wrong return value was added 6 years ago in commit 72cf8dd8, andd apparently, nobody ever noticed.

zeroconf/Bonjour: fix OnSocketReady() return value
df33a898 · Max Kellermann · 325c7b8e · df33a898 · df33a898
Commit df33a898 authored Apr 04, 2019 by Max Kellermann
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

NEWS NEWS +1 -0

ZeroconfBonjour.cxx src/zeroconf/ZeroconfBonjour.cxx +1 -1

No files found.
--- a/NEWS
+++ b/NEWS
 ver 0.21.8 (not yet released)
 * output
  - httpd: add missing mutex lock
+* fix Bonjour bug
 * fix build failure with GCC 9

 ver 0.21.7 (2019/04/03)

--- a/src/zeroconf/ZeroconfBonjour.cxx
+++ b/src/zeroconf/ZeroconfBonjour.cxx
@@ -50,7 +50,7 @@ protected:
 	/* virtual methods from class SocketMonitor */
 	bool OnSocketReady(gcc_unused unsigned flags) noexcept override {
 		DNSServiceProcessResult(service_ref);
-		return false;
+		return true;
 	}
 };