From: Nicolas Pitre <nico@fluxnic.net>
Date: Tue, 2 Feb 2010 05:05:42 +0000 (-0500)
Subject: ARM semihosting: fix EOF handling with SYS_READ
X-Git-Tag: v0.4.0-rc2~24
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=3d2d5dcc9c27b84dc2e5e9ed53be0f784a450042;p=openocd

ARM semihosting: fix EOF handling with SYS_READ

The semihosting interface has a strange convention for read/write where
the unused amount of buffer must be returned.  We failed to return the
total buffer size when the local read() call returned 0.

Signed-off-by: Nicolas Pitre <nico@marvell.com>
---

diff --git a/src/target/arm_semihosting.c b/src/target/arm_semihosting.c
index c41c5a00..9b853d9f 100644
--- a/src/target/arm_semihosting.c
+++ b/src/target/arm_semihosting.c
@@ -190,7 +190,7 @@ static int do_semihosting(struct target *target)
 			} else {
 				result = read(fd, buf, l);
 				armv4_5->semihosting_errno = errno;
-				if (result > 0) {
+				if (result >= 0) {
 					retval = target_write_buffer(target, a, result, buf);
 					if (retval != ERROR_OK) {
 						free(buf);