diff options
author | Spencer Oliver <spen@spen-soft.co.uk> | 2013-02-25 15:45:12 +0000 |
---|---|---|
committer | Freddie Chopin <freddie.chopin@gmail.com> | 2013-03-06 20:24:24 +0000 |
commit | 1d1f1b95a7516af3ce845c79af73511c82035528 (patch) | |
tree | 6bf47ec7932c7726d28245937b50dd0fe6a97e4d | |
parent | 0581ab7855422eea727892fb4aed7312c498c760 (diff) |
ti-icdi: catch failed icdi_send_cmd
warnings detected by clang.
Change-Id: I1532bcc12a8ab7446646dfb2a7afa8894ff03679
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/1180
Tested-by: jenkins
Reviewed-by: Xiaofan <xiaofanc@gmail.com>
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
-rw-r--r-- | src/jtag/drivers/ti_icdi_usb.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/jtag/drivers/ti_icdi_usb.c b/src/jtag/drivers/ti_icdi_usb.c index 67295223..dcffd2fb 100644 --- a/src/jtag/drivers/ti_icdi_usb.c +++ b/src/jtag/drivers/ti_icdi_usb.c @@ -346,6 +346,8 @@ static int icdi_usb_query(void *handle) h = (struct icdi_usb_handle_s *)handle; result = icdi_send_cmd(handle, "qSupported"); + if (result != ERROR_OK) + return result; /* check result */ result = icdi_get_cmd_result(handle); @@ -412,6 +414,8 @@ static int icdi_usb_run(void *handle) /* resume target at current address */ result = icdi_send_cmd(handle, "c"); + if (result != ERROR_OK) + return result; /* check result */ result = icdi_get_cmd_result(handle); @@ -429,6 +433,8 @@ static int icdi_usb_halt(void *handle) /* this query halts the target ?? */ result = icdi_send_cmd(handle, "?"); + if (result != ERROR_OK) + return result; /* check result */ result = icdi_get_cmd_result(handle); @@ -446,6 +452,8 @@ static int icdi_usb_step(void *handle) /* step target at current address */ result = icdi_send_cmd(handle, "s"); + if (result != ERROR_OK) + return result; /* check result */ result = icdi_get_cmd_result(handle); |