diff options
| author | Andreas Schneider <asn@samba.org> | 2014-04-10 10:30:40 +0200 |
|---|---|---|
| committer | Andreas Schneider <asn@samba.org> | 2014-04-14 15:28:56 +0200 |
| commit | f165d65e82eaaf139a89cd8b3f58fbe60f8cbecd (patch) | |
| tree | b73e32013075f9290025610e18f12b9af8fd6fe4 | |
| parent | 2254633003312b1f7462a408480483a2ec58d18a (diff) | |
| download | socket_wrapper-f165d65e82eaaf139a89cd8b3f58fbe60f8cbecd.tar.gz socket_wrapper-f165d65e82eaaf139a89cd8b3f58fbe60f8cbecd.tar.xz socket_wrapper-f165d65e82eaaf139a89cd8b3f58fbe60f8cbecd.zip | |
echo_srv: Fix resource leak of s2 on error.
Found by Coverity.
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Simo Sorce <idra@samba.org>
| -rw-r--r-- | tests/echo_srv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/echo_srv.c b/tests/echo_srv.c index 537de08..4c65d19 100644 --- a/tests/echo_srv.c +++ b/tests/echo_srv.c @@ -307,11 +307,11 @@ static int socket_dup(int s) /* Test dup2 */ s = dup2(s2, s); + close(s2); if (s == -1) { perror("dup"); return -1; } - close(s2); ZERO_STRUCT(srv_ss3); srv_ss3_len = sizeof(srv_ss3); |