fix(network): Avoid returning zero if connection is not dropped (#10640)

lucasssvaz · NPaolini · web-flow · commit 1712fb5eeee9 · 2024-11-25T12:17:38.000+02:00
Co-authored-by: NPaolini &lt;75562717+NPaolini@users.noreply.github.com&gt;
diff --git a/libraries/Network/src/NetworkClient.cpp b/libraries/Network/src/NetworkClient.cpp
@@ -549,7 +549,7 @@ uint8_t NetworkClient::connected() {
   }
   if (_connected) {
     uint8_t dummy;
-    int res = recv(fd(), &dummy, 0, MSG_DONTWAIT);
+    int res = recv(fd(), &dummy, 1, MSG_DONTWAIT | MSG_PEEK);
     // avoid unused var warning by gcc
     (void)res;
     // recv only sets errno if res is <= 0

Original file line number	Diff line number	Diff line change
`@@ -549,7 +549,7 @@ uint8_t NetworkClient::connected() {`
`549`	`549`	`}`
`550`	`550`	`if (_connected) {`
`551`	`551`	`uint8_t dummy;`
`552`		`- int res = recv(fd(), &dummy, 0, MSG_DONTWAIT);`
	`552`	`+ int res = recv(fd(), &dummy, 1, MSG_DONTWAIT \| MSG_PEEK);`
`553`	`553`	`// avoid unused var warning by gcc`
`554`	`554`	`(void)res;`
`555`	`555`	`// recv only sets errno if res is <= 0`