On 6/15/20 12:00 PM, kon...@adacore.com wrote:
> From: KONRAD Frederic <frederic.kon...@adacore.com>
>
> Don't send the trailing 0 from the string.
>
Fixes: a331c6d7741
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
> Signed-off-by: KONRAD Frederic <frederic.kon...@adacore.com>
> ---
> hw/semihosting/console.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/hw/semihosting/console.c b/hw/semihosting/console.c
> index 22e7827..9b4fee9 100644
> --- a/hw/semihosting/console.c
> +++ b/hw/semihosting/console.c
> @@ -52,7 +52,9 @@ static GString *copy_user_string(CPUArchState *env,
> target_ulong addr)
>
> do {
> if (cpu_memory_rw_debug(cpu, addr++, &c, 1, 0) == 0) {
> - s = g_string_append_c(s, c);
> + if (c) {
> + s = g_string_append_c(s, c);
> + }
> } else {
> qemu_log_mask(LOG_GUEST_ERROR,
> "%s: passed inaccessible address " TARGET_FMT_lx,
>
