Commit ceebb41d authored by Andrew Vagin's avatar Andrew Vagin Committed by Pavel Emelyanov

files: don't leak a file descriptor on an error path

CID 84652 (#1 of 1): Resource leak (RESOURCE_LEAK)
6. leaked_handle: Handle variable newfd going out of scope leaks the handle.
Signed-off-by: 's avatarAndrew Vagin <avagin@openvz.org>
Signed-off-by: 's avatarPavel Emelyanov <xemul@parallels.com>
parent 0668e029
...@@ -1526,6 +1526,7 @@ int inherit_fd_resolve_clash(int fd) ...@@ -1526,6 +1526,7 @@ int inherit_fd_resolve_clash(int fd)
} }
if (close(fd) == -1) { if (close(fd) == -1) {
close(newfd);
pr_perror("Can't close inherit fd %d", fd); pr_perror("Can't close inherit fd %d", fd);
return -1; return -1;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment