diff options
author | Andreas Schneider <asn@samba.org> | 2014-04-10 10:46:18 +0200 |
---|---|---|
committer | Andreas Schneider <asn@samba.org> | 2014-04-14 15:29:06 +0200 |
commit | fab60bb3f61c7dfd1f028a1f9821dab9ad826df7 (patch) | |
tree | cb1b0c3d56610b39e9c0def09a913ce782a9b98c /tests/test_echo_tcp_socket_options.c | |
parent | efd810d5e12e7b1409542874f5bf29ab0d431a7b (diff) | |
download | socket_wrapper-fab60bb3f61c7dfd1f028a1f9821dab9ad826df7.tar.gz socket_wrapper-fab60bb3f61c7dfd1f028a1f9821dab9ad826df7.tar.xz socket_wrapper-fab60bb3f61c7dfd1f028a1f9821dab9ad826df7.zip |
tests: Fix a possible resource leak on error in sockopt test.
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Simo Sorce <idra@samba.org>
Diffstat (limited to 'tests/test_echo_tcp_socket_options.c')
-rw-r--r-- | tests/test_echo_tcp_socket_options.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/test_echo_tcp_socket_options.c b/tests/test_echo_tcp_socket_options.c index 8ec631f..49b1187 100644 --- a/tests/test_echo_tcp_socket_options.c +++ b/tests/test_echo_tcp_socket_options.c @@ -118,7 +118,6 @@ static void test_bind_ipv6_only(void **state) case AF_INET: assert_int_equal(rc, -1); - close(s); break; case AF_INET6: assert_int_equal(rc, 0); @@ -126,11 +125,12 @@ static void test_bind_ipv6_only(void **state) rc = bind(s, ri->ai_addr, ri->ai_addrlen); assert_int_equal(rc, 0); - close(s); break; default: break; } + + close(s); } freeaddrinfo(res); } |