You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2015/02/05 16:00:08 UTC
[2/2] trafficserver git commit: TS-3287: Close socket to prevent
resource leak
TS-3287: Close socket to prevent resource leak
Coverity CID #1022129
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/7ac50d86
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/7ac50d86
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/7ac50d86
Branch: refs/heads/master
Commit: 7ac50d860015e1cce1e7b3d19d4eb07d42a583d8
Parents: 20b25d6
Author: Phil Sorber <so...@apache.org>
Authored: Thu Feb 5 07:58:37 2015 -0700
Committer: Phil Sorber <so...@apache.org>
Committed: Thu Feb 5 07:59:44 2015 -0700
----------------------------------------------------------------------
tools/jtest/jtest.cc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/7ac50d86/tools/jtest/jtest.cc
----------------------------------------------------------------------
diff --git a/tools/jtest/jtest.cc b/tools/jtest/jtest.cc
index 046c26f..7f03971 100644
--- a/tools/jtest/jtest.cc
+++ b/tools/jtest/jtest.cc
@@ -1401,8 +1401,10 @@ static int open_server(unsigned short int port, accept_fn_t accept_fn) {
exit( EXIT_FAILURE);
}
if ((err=bind (sock, (struct sockaddr *) &name, sizeof (name))) < 0) {
- if (errno == EADDRINUSE)
+ if (errno == EADDRINUSE) {
+ close(sock);
return -EADDRINUSE;
+ }
perror("bind");
exit( EXIT_FAILURE);
}