From: Øyvind Harboe <oyvind.harboe@zylin.com>
Date: Tue, 8 Nov 2011 16:50:01 +0000 (+0100)
Subject: buspirate: add missing error propagation
X-Git-Tag: v0.6.0-rc1~443
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=7d9d056eaee3d118a9dc1628692249522e769bb4;p=openocd

buspirate: add missing error propagation

found by clang.

Change-Id: I80ea8e6afc8dcc1aa7edb6f63af0d94f6781b81c
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Reviewed-on: http://openocd.zylin.com/182
Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
---

diff --git a/src/jtag/drivers/buspirate.c b/src/jtag/drivers/buspirate.c
index 3a368ebb..41e3e194 100644
--- a/src/jtag/drivers/buspirate.c
+++ b/src/jtag/drivers/buspirate.c
@@ -608,6 +608,10 @@ static int buspirate_tap_execute(void)
 	}
 
 	ret = buspirate_serial_read(buspirate_fd, tmp, bytes_to_send + 3);
+	if (ret != bytes_to_send + 3) {
+		LOG_ERROR("error reading");
+		return ERROR_FAIL;
+	}
 	in_buf = (uint8_t *)(&tmp[3]);
 
 	/* parse the scans */
@@ -743,6 +747,10 @@ static void buspirate_jtag_enable(int fd)
 				cmd_sent = 1;
 				tmp[0] = CMD_ENTER_OOCD;
 				ret = buspirate_serial_write(fd, tmp, 1);
+				if (ret != 1) {
+					LOG_ERROR("error reading");
+					exit(-1);
+				}
 			}
 		} else if (strncmp(tmp, "OCD1", 4) == 0)
 			done = 1;