On 4/9/20 2:15 PM, Alex Bennée wrote:
> In the original bug report long files names in Guix caused
> /proc/self/stat be truncated without the trailing ") " as specified in
> proc manpage which says:
>     (2) comm  %s
>            The  filename of the executable, in parentheses.  This
>            is visible whether or not the  executable  is  swapped
>            out.
> 
> Additionally it should only be reporting the executable name rather
> than the full path. Fix both these failings while cleaning up the code
> to use GString to build up the reported values. As the whole function
> is cleaned up also adjust the white space to the current coding style.
> 
> Message-ID: <fb4c55fa-d539-67ee-c6c9-de8fb63c8...@inria.fr>
> Reported-by: Brice Goglin <brice.gog...@inria.fr>
> Cc: Philippe_Mathieu-Daudé <phi...@redhat.com>
> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
> ---
>  linux-user/syscall.c | 43 +++++++++++++++++++------------------------
>  1 file changed, 19 insertions(+), 24 deletions(-)

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>


r~


Reply via email to