From: Spencer Oliver <spen@spen-soft.co.uk>
Date: Mon, 9 Jan 2012 21:35:19 +0000 (+0000)
Subject: target: fix missing semihosting return path
X-Git-Tag: v0.6.0-rc1~323
X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=commitdiff_plain;h=cd22bedf6fd7202a3c788184aa08e0ae2e7b662f

target: fix missing semihosting return path

bug nicely caught by clang.

Change-Id: I7abf0fdd76666fb3eb1c83e3edfd01e0da485ffe
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/359
Tested-by: jenkins
Reviewed-by: Øyvind Harboe <oyvindharboe@gmail.com>
---

diff --git a/src/target/armv4_5.c b/src/target/armv4_5.c
index 706733d211..7468e389f4 100644
--- a/src/target/armv4_5.c
+++ b/src/target/armv4_5.c
@@ -966,6 +966,7 @@ COMMAND_HANDLER(handle_arm_semihosting_command)
 	if (!arm->setup_semihosting)
 	{
 		command_print(CMD_CTX, "semihosting not supported for current target");
+		return ERROR_FAIL;
 	}
 
 	if (CMD_ARGC > 0)