From 02bac862c003b2da5bb5c396e3ca19a7fb6990ea Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Wed, 3 Nov 2010 11:58:25 +0000 Subject: 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 Signed-off-by: Ian Jackson --- tools/debugger/gdbsx/gx/gx_comm.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'tools/debugger') 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"); -- cgit v1.2.3