pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmocore/+/35231?usp=email )
Change subject: tests/socket: Avoid keeping unneeded sockets open during next test cases
......................................................................
tests/socket: Avoid keeping unneeded sockets open during next test cases
Change-Id: I59cde0843545428e2bc490cc9470b76cbd7f8393
---
M tests/socket/socket_sctp_test.c
1 file changed, 12 insertions(+), 0 deletions(-)
Approvals:
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/tests/socket/socket_sctp_test.c b/tests/socket/socket_sctp_test.c
index e099be9..5948abc 100644
--- a/tests/socket/socket_sctp_test.c
+++ b/tests/socket/socket_sctp_test.c
@@ -107,6 +107,7 @@
addrv4_rem, addrv4_size, listen_port_v4,
OSMO_SOCK_F_BIND|OSMO_SOCK_F_CONNECT);
OSMO_ASSERT(fd >= 0);
+ close(fd);
printf("Checking osmo_sock_init2_multiaddr(AF_UNSPEC) must fail on mixed IPv4 &
IPv6\n");
fd = osmo_sock_init2_multiaddr(AF_UNSPEC, SOCK_STREAM, IPPROTO_SCTP,
@@ -128,6 +129,7 @@
addrv4_rem, addrv4_size, listen_port_v4,
OSMO_SOCK_F_BIND|OSMO_SOCK_F_CONNECT);
OSMO_ASSERT(fd >= 0);
+ close(fd);
printf("Checking osmo_sock_init2_multiaddr(AF_UNSPEC) BIND + CONNECT on
IPv6\n");
fd = osmo_sock_init2_multiaddr(AF_UNSPEC, SOCK_STREAM, IPPROTO_SCTP,
@@ -135,6 +137,7 @@
addrv6_rem, addrv6_size, listen_port_v6,
OSMO_SOCK_F_BIND|OSMO_SOCK_F_CONNECT);
OSMO_ASSERT(fd >= 0);
+ close(fd);
close(listen_fd_v4);
close(listen_fd_v6);
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/35231?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I59cde0843545428e2bc490cc9470b76cbd7f8393
Gerrit-Change-Number: 35231
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged