[Spice-devel] [PATCH spice-server 2/2] reds-stream: Remove useless debug

Jonathon Jongsma jjongsma at redhat.com
Wed Oct 18 15:05:27 UTC 2017


Acked-by: Jonathon Jongsma <jjongsma at redhat.com>


On Wed, 2017-10-18 at 07:30 +0100, Frediano Ziglio wrote:
> There are already other debugging code showing channel closure.
> Not closed file descriptors can easily be detected with other
> tools (like netstat or /proc file system).
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  server/reds-stream.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/server/reds-stream.c b/server/reds-stream.c
> index c12893793..0ba464654 100644
> --- a/server/reds-stream.c
> +++ b/server/reds-stream.c
> @@ -367,7 +367,6 @@ void reds_stream_free(RedsStream *s)
>      }
>  
>      reds_stream_remove_watch(s);
> -    spice_debug("close socket fd %d", s->socket);
>      close(s->socket);
>  
>      g_free(s);


More information about the Spice-devel mailing list