[libvirt] [PATCH] Use virNetServerClientImmediateClose() rather than virNetServerClientClose()

Ján Tomko jtomko at redhat.com
Mon Nov 25 12:16:47 UTC 2019


On Mon, Nov 25, 2019 at 07:23:07PM +0800, LanceLiu wrote:

Please include a description why this change is needed in the commit
message, to give the people looking at the git history some
understanding of it without having to go through the mailing list
archives.

See the following blog post for some commit message guidelines:
https://www.berrange.com/posts/2012/06/27/thoughts-on-improving-openstack-git-commit-practicehistory/

>---
> src/remote/remote_daemon_stream.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>

>@@ -262,7 +262,7 @@ daemonStreamEvent(virStreamPtr st, int events, void *opaque)
>         }
>         daemonRemoveClientStream(client, stream);
>         if (ret < 0)
>-            virNetServerClientClose(client);
>+        	virNetServerClientImmediateClose(client);

Looks like the indentation is off here.

Jano

>         goto cleanup;
>     }
>
>-- 
>1.8.3.1
>
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20191125/d7a3acf4/attachment-0001.sig>


More information about the libvir-list mailing list