[PATCH] libosmo-netif[master]: osmo_stream_srv_fd_cb(): don't leak socket FDs on errors

[Neels Hofmeyr]

Review at  https://gerrit.osmocom.org/1320

osmo_stream_srv_fd_cb(): don't leak socket FDs on errors

So far we seem to assume that the accept_cb does all handling of socket fd
cleanup. However, there are cases where there is no accept_cb set, the
accept_cb returns error, or an earlier sctp_sock_activate_events() fails.

For those cases, close the socket and return an error code.

Fixes: CID#57915
Change-Id: I3a3ce9194ab7ca5c1921fc79c2a1c9e10a552cf0
---
M src/stream.c
1 file changed, 18 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/20/1320/1

diff --git a/src/stream.c b/src/stream.c
index 7d40df2..baede42 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -391,6 +391,7 @@
 static int osmo_stream_srv_fd_cb(struct osmo_fd *ofd, unsigned int what)
 {
 	int ret;
+	int sock_fd;
 	struct sockaddr_in sa;
 	socklen_t sa_len = sizeof(sa);
 	struct osmo_stream_srv_link *link = ofd->data;
@@ -403,14 +404,27 @@
 	}
 	LOGP(DLINP, LOGL_DEBUG, "accept()ed new link from %s to port %u\n",
 		inet_ntoa(sa.sin_addr), link->port);
+	sock_fd = ret;
 
-	if (link->proto == IPPROTO_SCTP)
-		sctp_sock_activate_events(ret);
+	if (link->proto == IPPROTO_SCTP) {
+		ret = sctp_sock_activate_events(sock_fd);
+		if (ret < 0)
+			goto error_close_socket;
+	}
 
-	if (link->accept_cb)
-		link->accept_cb(link, ret);
+	if (!link->accept_cb) {
+		ret = -ENOTSUP;
+		goto error_close_socket;
+	}
 
+	ret = link->accept_cb(link, sock_fd);
+	if (ret)
+		goto error_close_socket;
 	return 0;
+
+error_close_socket:
+	close(sock_fd);
+	return ret;
 }
 
 struct osmo_stream_srv_link *osmo_stream_srv_link_create(void *ctx)

-- 
To view, visit https://gerrit.osmocom.org/1320
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3a3ce9194ab7ca5c1921fc79c2a1c9e10a552cf0
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>