SocketWrapper: fix sock_fd check condition

Author: leonardocavagnis

libraries/SocketWrapper/SocketWrapper.h

@@ -73,7 +73,7 @@ class ZephyrSocketWrapper {
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
 		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
 									   socket_deleter());
-		if (!sock_fd || *sock_fd < 0) {
+		if (!sock_fd) {
 			rv = false;
 
 			goto exit;
@@ -107,7 +107,7 @@ class ZephyrSocketWrapper {
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
 		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
 									   socket_deleter());
-		if (!sock_fd || *sock_fd < 0) {
+		if (!sock_fd) {
 			return false;
 		}
 
@@ -168,7 +168,7 @@ class ZephyrSocketWrapper {
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TLS_1_2);
 		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
 									   socket_deleter());
-		if (!sock_fd || *sock_fd < 0) {
+		if (!sock_fd) {
 			goto exit;
 		}
 
@@ -246,7 +246,7 @@ class ZephyrSocketWrapper {
 	}
 
 	void close() {
-		if (sock_fd && *sock_fd != -1) {
+		if (sock_fd) {
 			sock_fd = nullptr;
 		}
 	}
@@ -261,7 +261,7 @@ class ZephyrSocketWrapper {
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
 		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
 									   socket_deleter());
-		if (!sock_fd || *sock_fd < 0) {
+		if (!sock_fd) {
 			return false;
 		}
 

libraries/SocketWrapper/ZephyrClient.h

@@ -104,7 +104,7 @@ class ZephyrClient : public arduino::Client, ZephyrSocketWrapper {
 	}
 
 	operator bool() {
-		return sock_fd && *sock_fd != -1;
+		return sock_fd;
 	}
 
 	String remoteIP() {

libraries/SocketWrapper/ZephyrServer.h

@@ -42,7 +42,7 @@ class ZephyrServer : public arduino::Server, ZephyrSocketWrapper {
 	}
 
 	explicit operator bool() {
-		return sock_fd && *sock_fd != -1;
+		return sock_fd;
 	}
 
 	ZephyrClient accept(uint8_t *status = nullptr) {