[PATCH] xattr: use printstr_ex instead of print_quoted_string

Line Printer lineprinter0 at gmail.com
Thu Dec 22 11:13:52 UTC 2016


On Thu, Dec 22, 2016 at 1:16 PM, Gabriel Laskar <gabriel at lse.epita.fr> wrote:
> This fixes the display when using `-s` to limit the string size of the
> values displayed by {get,set}xattr.
>
> Signed-off-by: Gabriel Laskar <gabriel at lse.epita.fr>
> Reported-by: Марк Коренберг <socketpair at gmail.com>
>
> * xattr.c (print_xattr_val): use printstr_ex instead of
> print_quoted_string
>
> Signed-off-by: Gabriel Laskar <gabriel at lse.epita.fr>
> ---
>  xattr.c | 13 +++----------
>  1 file changed, 3 insertions(+), 10 deletions(-)

Could you also add a testcase to tests/xattr.test?

> diff --git a/xattr.c b/xattr.c
> index 30a8467c..8e22f7eb 100644
> --- a/xattr.c
> +++ b/xattr.c
> @@ -45,19 +45,12 @@ print_xattr_val(struct tcb *tcp,
>                 unsigned long insize,
>                 unsigned long size)
>  {
> -       static char buf[XATTR_SIZE_MAX];
> -
>         tprints(", ");
>
> -       if (!addr || size > sizeof(buf))
> +       if (!addr || size > XATTR_SIZE_MAX)
>                 printaddr(addr);
> -       else if (!size || !umoven_or_printaddr(tcp, addr, size, buf)) {
> -               /* Don't print terminating NUL if there is one. */
> -               if (size && buf[size - 1] == '\0')
> -                       --size;
> -
> -               print_quoted_string(buf, size, 0);
> -       }
> +       else
> +               printstr_ex(tcp, addr, size, QUOTE_OMIT_TRAILING_0);
>         tprintf(", %lu", insize);
>  }
>
> --
> 2.11.0
>
>
> ------------------------------------------------------------------------------
> Developer Access Program for Intel Xeon Phi Processors
> Access to Intel Xeon Phi processor-based developer platforms.
> With one year of Intel Parallel Studio XE.
> Training and support from Colfax.
> Order your platform today.http://sdm.link/intel
> _______________________________________________
> Strace-devel mailing list
> Strace-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/strace-devel



-- 
Elvira Khabirova




More information about the Strace-devel mailing list