net: zperf: Fix invalid zsock_recv() error check

rlubos · kartben · commit 02530beedac0 · 2025-05-23T17:20:10.000+02:00
The return value can only be -1, errno value should be verified instead
for the actual error code.

Signed-off-by: Robert Lubos &lt;robert.lubos@nordicsemi.no&gt;
diff --git a/subsys/net/lib/zperf/zperf_udp_uploader.c b/subsys/net/lib/zperf/zperf_udp_uploader.c
@@ -118,7 +118,7 @@ static inline int zperf_upload_fin(int sock,
 			}
 
 			ret = zsock_recv(sock, stats, sizeof(stats), 0);
-			if (ret == -EAGAIN) {
+			if (ret < 0 && errno == EAGAIN) {
 				NET_WARN("Stats receive timeout");
 			} else if (ret < 0) {
 				NET_ERR("Failed to receive packet (%d)", errno);

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ static inline int zperf_upload_fin(int sock,`
`118`	`118`	`}`
`119`	`119`
`120`	`120`	`ret = zsock_recv(sock, stats, sizeof(stats), 0);`
`121`		`- if (ret == -EAGAIN) {`
	`121`	`+ if (ret < 0 && errno == EAGAIN) {`
`122`	`122`	`NET_WARN("Stats receive timeout");`
`123`	`123`	`} else if (ret < 0) {`
`124`	`124`	`NET_ERR("Failed to receive packet (%d)", errno);`