Fix WiFiClient error handling (espressif#1388)

chemicstry · me-no-dev · commit 611db61d8490 · 2018-05-14T13:01:22.000+02:00
diff --git a/libraries/WiFi/src/WiFiClient.cpp b/libraries/WiFi/src/WiFiClient.cpp
@@ -313,7 +313,7 @@ uint8_t WiFiClient::connected()
     if (_connected) {
         uint8_t dummy;
         int res = recv(fd(), &dummy, 0, MSG_DONTWAIT);
-        if (res <= 0) {
+        if (res < 0) {
             switch (errno) {
                 case ENOTCONN:
                 case EPIPE:
@@ -328,7 +328,6 @@ uint8_t WiFiClient::connected()
             }
         }
         else {
-            // Should never happen since requested 0 bytes
             _connected = true;
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -313,7 +313,7 @@ uint8_t WiFiClient::connected()`
`313`	`313`	`if (_connected) {`
`314`	`314`	`uint8_t dummy;`
`315`	`315`	`int res = recv(fd(), &dummy, 0, MSG_DONTWAIT);`
`316`		`- if (res <= 0) {`
	`316`	`+ if (res < 0) {`
`317`	`317`	`switch (errno) {`
`318`	`318`	`case ENOTCONN:`
`319`	`319`	`case EPIPE:`
`@@ -328,7 +328,6 @@ uint8_t WiFiClient::connected()`
`328`	`328`	`}`
`329`	`329`	`}`
`330`	`330`	`else {`
`331`		`- // Should never happen since requested 0 bytes`
`332`	`331`	`_connected = true;`
`333`	`332`	`}`
`334`	`333`	`}`