Update libraries/SocketWrapper/SocketWrapper.h

leonardocavagnis · andreagilardoni · web-flow · commit 231edceae904 · 2025-11-12T10:31:24.000+01:00
Co-authored-by: Andrea Gilardoni &lt;4046444+andreagilardoni@users.noreply.github.com&gt;
diff --git a/libraries/SocketWrapper/SocketWrapper.h b/libraries/SocketWrapper/SocketWrapper.h
@@ -29,7 +29,7 @@ class ZephyrSocketWrapper {
 public:
 	ZephyrSocketWrapper() = default;
 
-	ZephyrSocketWrapper(int fd) : sock_fd(std::shared_ptr<int>(new int(fd), socket_deleter())) {
+	ZephyrSocketWrapper(int fd) : sock_fd(std::shared_ptr<int>(fd<0 ? nullptr : new int(fd), socket_deleter())) {
 	}
 
 	~ZephyrSocketWrapper() = default; // socket close managed by shared_ptr

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ class ZephyrSocketWrapper {`
`29`	`29`	`public:`
`30`	`30`	`ZephyrSocketWrapper() = default;`
`31`	`31`
`32`		`- ZephyrSocketWrapper(int fd) : sock_fd(std::shared_ptr<int>(new int(fd), socket_deleter())) {`
	`32`	`+ ZephyrSocketWrapper(int fd) : sock_fd(std::shared_ptr<int>(fd<0 ? nullptr : new int(fd), socket_deleter())) {`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`~ZephyrSocketWrapper() = default; // socket close managed by shared_ptr`