net: sockets: Fix net pkt leak
If net_context_recv() returns a error, net pkt will not be released. For
example, net_context_recv() returns -EBADF because the TCP connection is
closed by the peer.
Handle the return value instead of using SET_ERRNO().
Signed-off-by: Aska Wu <aska.wu@linaro.org>
diff --git a/subsys/net/lib/sockets/sockets.c b/subsys/net/lib/sockets/sockets.c
index dd7d0b8..f3fe746 100644
--- a/subsys/net/lib/sockets/sockets.c
+++ b/subsys/net/lib/sockets/sockets.c
@@ -258,7 +258,12 @@
/* Register the callback before sending in order to receive the response
* from the peer.
*/
- SET_ERRNO(net_context_recv(ctx, zsock_received_cb, K_NO_WAIT, NULL));
+ err = net_context_recv(ctx, zsock_received_cb, K_NO_WAIT, NULL);
+ if (err < 0) {
+ net_pkt_unref(send_pkt);
+ errno = -err;
+ return -1;
+ }
if (dest_addr) {
err = net_context_sendto(send_pkt, dest_addr, addrlen, NULL,
