diff options
author | Keir Fraser <keir@xen.org> | 2010-11-08 15:39:37 +0000 |
---|---|---|
committer | Keir Fraser <keir@xen.org> | 2010-11-08 15:39:37 +0000 |
commit | b62d7a084709c07b58264d0086c73105261a3303 (patch) | |
tree | d0e927b2a8655387cd07ec855d8b3280e47e6caf /tools | |
parent | c39abc43138cdd9cc0e8baaf704acbf8c314d6cb (diff) | |
download | xen-b62d7a084709c07b58264d0086c73105261a3303.tar.gz xen-b62d7a084709c07b58264d0086c73105261a3303.tar.bz2 xen-b62d7a084709c07b58264d0086c73105261a3303.zip |
tools: gdbsx: Check return of write()
Not checking leads to warn_unused_result checks triggering in some
libraries and compilers. Combined with -Werror this breaks the build.
Signed-off-by: Gianni Tedesco <gianni.tedesco@citrix.com>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
xen-unstable changeset: 22349:3f98881703aa
xen-unstable date: Wed Nov 03 11:58:25 2010 +0000
Diffstat (limited to 'tools')
-rw-r--r-- | tools/debugger/gdbsx/gx/gx_comm.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/tools/debugger/gdbsx/gx/gx_comm.c b/tools/debugger/gdbsx/gx/gx_comm.c index 156a9f5f55..ed4a7bef0e 100644 --- a/tools/debugger/gdbsx/gx/gx_comm.c +++ b/tools/debugger/gdbsx/gx/gx_comm.c @@ -227,13 +227,19 @@ gx_getpkt (char *buf) gxprt("Bad checksum, sentsum=0x%x, csum=0x%x, buf=%s\n", (c1 << 4) + c2, csum, buf); - write(remote_fd, "-", 1); + if (write(remote_fd, "-", 1) != 1) { + perror("write"); + return -1; + } } if (gx_remote_dbg) { gxprt("getpkt (\"%s\"); [sending ack] \n", buf); } - write(remote_fd, "+", 1); + if (write(remote_fd, "+", 1) != 1) { + perror("write"); + return -1; + } if (gx_remote_dbg) { gxprt("[sent ack]\n"); |