diff options
author | kfraser@localhost.localdomain <kfraser@localhost.localdomain> | 2007-07-24 18:28:48 +0100 |
---|---|---|
committer | kfraser@localhost.localdomain <kfraser@localhost.localdomain> | 2007-07-24 18:28:48 +0100 |
commit | 698543a3139d11d724d2b7b0622a7ea42051a696 (patch) | |
tree | 0aa2dbd1eb56627c5d7a060b8bb7701f3f378498 /tools/xenstore/xenstored_watch.c | |
parent | e313dc3ed8a13242ce9a9c5cb658d482c44a192e (diff) | |
download | xen-698543a3139d11d724d2b7b0622a7ea42051a696.tar.gz xen-698543a3139d11d724d2b7b0622a7ea42051a696.tar.bz2 xen-698543a3139d11d724d2b7b0622a7ea42051a696.zip |
xenstore: Small cleanups and fixes.
1. readfd/writefd account for EINTR/EAGAIN errno returns.
2. Handle zero return from ->read() and ->write() handlers
symmetrically.
3. Fix some indentation issues (use hard tabs).
Signed-off-by: Keir Fraser <keir@xensource.com>
Diffstat (limited to 'tools/xenstore/xenstored_watch.c')
-rw-r--r-- | tools/xenstore/xenstored_watch.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/tools/xenstore/xenstored_watch.c b/tools/xenstore/xenstored_watch.c index 27e082a24f..f5692f83e5 100644 --- a/tools/xenstore/xenstored_watch.c +++ b/tools/xenstore/xenstored_watch.c @@ -73,11 +73,10 @@ static void add_event(struct connection *conn, data = talloc_array(watch, char, len); strcpy(data, name); strcpy(data + strlen(name) + 1, watch->token); - send_reply(conn, XS_WATCH_EVENT, data, len); + send_reply(conn, XS_WATCH_EVENT, data, len); talloc_free(data); } -/* FIXME: we fail to fire on out of memory. Should drop connections. */ void fire_watches(struct connection *conn, const char *name, bool recurse) { struct connection *i; @@ -130,7 +129,7 @@ void do_watch(struct connection *conn, struct buffered_data *in) /* Check for duplicates. */ list_for_each_entry(watch, &conn->watches, list) { if (streq(watch->node, vec[0]) && - streq(watch->token, vec[1])) { + streq(watch->token, vec[1])) { send_error(conn, EEXIST); return; } |