diff options
author | Øyvind Harboe <oyvind.harboe@zylin.com> | 2011-11-08 17:50:01 +0100 |
---|---|---|
committer | Spencer Oliver <spen@spen-soft.co.uk> | 2011-11-09 16:30:24 +0000 |
commit | 7d9d056eaee3d118a9dc1628692249522e769bb4 (patch) | |
tree | c43d54b1219b79173b275881ef5556b41f2c12e5 /src/jtag/drivers/buspirate.c | |
parent | 86d8ee7fe246f7cff50c13d3ca2a35dbf4ffff37 (diff) |
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>
Diffstat (limited to 'src/jtag/drivers/buspirate.c')
-rw-r--r-- | src/jtag/drivers/buspirate.c | 8 |
1 files changed, 8 insertions, 0 deletions
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; |