From 3db4e7f84cf795de8559ea0d96eaa491999ccf24 Mon Sep 17 00:00:00 2001 From: Bernhard Reutner-Fischer Date: Fri, 19 Oct 2018 15:25:41 +0200 Subject: printf: fix printing +-prefixed numbers Thanks to Cristian Ionescu-Idbohrn for noticing. Also fix "%d" ' 42' to skip leading whitespace. function old new delta print_direc 435 454 +19 bb_strtoll 99 103 +4 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 2/0 up/down: 23/0) Total: 23 bytes Signed-off-by: Bernhard Reutner-Fischer --- coreutils/printf.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'coreutils') diff --git a/coreutils/printf.c b/coreutils/printf.c index a666ff7ac..b2429c5cf 100644 --- a/coreutils/printf.c +++ b/coreutils/printf.c @@ -191,6 +191,7 @@ static void print_direc(char *format, unsigned fmt_length, if (have_width - 1 == have_prec) have_width = NULL; + /* multiconvert sets errno = 0, but %s needs it cleared */ errno = 0; switch (format[fmt_length - 1]) { @@ -199,7 +200,7 @@ static void print_direc(char *format, unsigned fmt_length, break; case 'd': case 'i': - llv = my_xstrtoll(argument); + llv = my_xstrtoll(skip_whitespace(argument)); print_long: if (!have_width) { if (!have_prec) @@ -217,7 +218,7 @@ static void print_direc(char *format, unsigned fmt_length, case 'u': case 'x': case 'X': - llv = my_xstrtoull(argument); + llv = my_xstrtoull(skip_whitespace(argument)); /* cheat: unsigned long and long have same width, so... */ goto print_long; case 's': -- cgit v1.2.3