kvm tools: Simply write_in_full() check semantics

write_in_full() would not return until count byes has been written or
error has occurred.

So

   if (write_in_full(fd, buf, count) < 0)
	goto err;

is enough.

And

   if (write_in_full(fd, buf, count) != count)
	goto err;

is not necessary.

Signed-off-by: Asias He <asias.hejun@gmail.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
This commit is contained in:
Asias He
2012-01-09 17:19:26 +08:00
committed by Will Deacon
parent fd5b45a1d3
commit ca08826821
+2 -2
View File
@@ -37,7 +37,7 @@ int kvm_ipc__send(int fd, u32 type)
{
struct kvm_ipc_head head = {.type = type, .len = 0,};
if (write_in_full(fd, &head, sizeof(head)) != sizeof(head))
if (write_in_full(fd, &head, sizeof(head)) < 0)
return -1;
return 0;
@@ -47,7 +47,7 @@ int kvm_ipc__send_msg(int fd, u32 type, u32 len, u8 *msg)
{
struct kvm_ipc_head head = {.type = type, .len = len,};
if (write_in_full(fd, &head, sizeof(head)) != sizeof(head))
if (write_in_full(fd, &head, sizeof(head)) < 0)
return -1;
if (write_in_full(fd, msg, len) < 0)