Make xenstored EOF-safe

As bug 968 suggests, xenstored must properly handle EOF from clients.

From: Pavel Kankovsky <[email protected]>

Signed-off-by Samuel Thibault <[email protected]>

diff -r 6fd17d0dcbcd tools/xenstore/xenstored_core.c

tools/xenstore/xenstored_core.c Tue Nov 27 14:56:35 2007 +0000 renamed from tools/xenstore/xenstored_core.c Tue Nov 27 12:49:16 2007 +0000 +2 -2

@@ -1266,7 +1266,7 @@ static void handle_input(struct connecti
 if (in->inhdr) {
 bytes = conn->read(conn, in->hdr.raw + in->used,
  sizeof(in->hdr) - in->used);
-  if (bytes < 0)
+  if (bytes <= 0)
 goto bad_client;
 in->used += bytes;
 if (in->used != sizeof(in->hdr))

@@ -1288,7 +1288,7 @@ static void handle_input(struct connecti
 
 bytes = conn->read(conn, in->buffer + in->used,
  in->hdr.msg.len - in->used);
- if (bytes < 0)
+ if (bytes <= 0)
 goto bad_client;
 
 in->used += bytes;

_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xensource.com/xen-devel

Xen-unstable 15643 introduced this bug, but it should have been fixed satisfactorily by 15644 I think?