diff options
| author | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2013-05-13 14:04:08 +0200 |
|---|---|---|
| committer | Danny Al-Gaaf <danny.al-gaaf@bisect.de> | 2013-05-14 18:39:31 +0200 |
| commit | c3c140b38ce1a5a5ad5e316c50707f68e6bfe2ab (patch) | |
| tree | eb2fdb126a1d27675690e2d8030314c2703507d9 | |
| parent | 043ea2ce9bbde0acb3d6dc3e51cafafba0220875 (diff) | |
| download | ceph-c3c140b38ce1a5a5ad5e316c50707f68e6bfe2ab.tar.gz | |
tools/ceph.cc: close file descriptor in error case
CID 717122 (#1 of 1): Resource leak (RESOURCE_LEAK)
leaked_handle: Handle variable "fd" going out of scope leaks
the handle.
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
| -rw-r--r-- | src/tools/ceph.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/tools/ceph.cc b/src/tools/ceph.cc index 2859c86b1de..fd4dbb17d7b 100644 --- a/src/tools/ceph.cc +++ b/src/tools/ceph.cc @@ -232,7 +232,8 @@ int do_admin_socket(string path, string cmd) if (connect(fd, (struct sockaddr *) &address, sizeof(struct sockaddr_un)) != 0) { cerr << "connect to " << path << " failed with " << cpp_strerror(errno) << std::endl; - return -1; + r = -1; + goto out; } char *buf; |