Re: [Qemu-devel] linux-user: fix getcwd syscall

[Aurelien Jarno]

On Thu, Apr 16, 2009 at 03:04:23PM +0200, Arnaud Patard wrote:
> 
> The patch called "prefer glibc over direct syscalls" (commit 7118) has
> replaced the getcwd syscall with a call to the glibc. With this change,
> the syscall is returning -1 in error case and 0 otherwise.
> This is problematic as the sys_getcwd syscall should return the number
> of bytes written to the buffer including the '\0'.
> 
> Signed-off-by: Arnaud Patard <address@hidden>
> ---

Thanks, applied.

> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index 74b41a8..f5875aa 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -293,7 +293,7 @@ static int sys_getcwd1(char *buf, size_t size)
>        /* getcwd() sets errno */
>        return (-1);
>    }
> -  return (0);
> +  return strlen(buf)+1;
>  }
>  
>  #ifdef CONFIG_ATFILE


-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
address@hidden                 http://www.aurel32.net