[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] virtfs-proxy-helper: Fix handle leak to mak
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] virtfs-proxy-helper: Fix handle leak to make Coverity happy |
Date: |
Thu, 27 Nov 2014 13:47:38 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
<address@hidden> writes:
> From: Gonglei <address@hidden>
>
> Coverity report:
> (94) Event open_fn: Returning handle opened by function "proxy_socket(char
> const *, uid_t, gid_t)". [details]
> (95) Event var_assign: Assigning: "sock" = handle returned from
> "proxy_socket(sock_name, own_u, own_g)".
> (103) Event leaked_handle: Handle variable "sock" going out of scope leaks
> the handle.
>
> Signed-off-by: Gonglei <address@hidden>
> ---
> fsdev/virtfs-proxy-helper.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/fsdev/virtfs-proxy-helper.c b/fsdev/virtfs-proxy-helper.c
> index c1da2d7..2d72def 100644
> --- a/fsdev/virtfs-proxy-helper.c
> +++ b/fsdev/virtfs-proxy-helper.c
> @@ -1150,6 +1150,9 @@ int main(int argc, char **argv)
>
> process_requests(sock);
> error:
> + if (sock_name && sock >= 0) {
> + close(sock);
> + }
> do_log(LOG_INFO, "Done\n");
> closelog();
> return 0;
Why if sock_name? What about sock gotten from -f?
If sock >= 0 is pointless, too, but needed to hush up Coverity.
[Qemu-devel] [PATCH 1/2] virtfs-proxy-helper: Fix possible socket leak., arei.gonglei, 2014/11/13
Re: [Qemu-devel] [PATCH 0/2] virtfs-proxy-helper: Fix Coverity warning, Gonglei, 2014/11/27