From 5f116629d80b66bd09d8dc2b849befb1e27cd21a Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Fri, 18 Jul 2008 18:41:55 +0000 Subject: printf: do not print garbage on "%Ld". closes bug 4214. function old new delta printf_main 633 637 +4 multiconvert 99 79 -20 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 1/1 up/down: 4/-20) Total: -16 bytes --- coreutils/printf.c | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) (limited to 'coreutils/printf.c') diff --git a/coreutils/printf.c b/coreutils/printf.c index ca035cc48..7f6235a87 100644 --- a/coreutils/printf.c +++ b/coreutils/printf.c @@ -63,11 +63,8 @@ typedef void FAST_FUNC (*converter)(const char *arg, void *result); static int multiconvert(const char *arg, void *result, converter convert) { - char s[sizeof(int)*3 + 2]; - if (*arg == '"' || *arg == '\'') { - sprintf(s, "%d", (unsigned char)arg[1]); - arg = s; + arg = utoa((unsigned char)arg[1]); } errno = 0; convert(arg, result); @@ -289,10 +286,22 @@ static char **print_formatted(char *f, char **argv) } } } + /* Remove size modifiers - "%Ld" would try to printf + * long long, we pass long, and it spews garbage */ if ((*f | 0x20) == 'l' || *f == 'h' || *f == 'z') { - ++f; - ++direc_length; + strcpy(f, f + 1); } +//FIXME: actually, the same happens with bare "%d": +//it printfs an int, but we pass long! +//What saves us is that on most arches stack slot +//is pointer-sized -> long-sized -> ints are promoted to longs +// for variadic functions -> printf("%d", int_v) is in reality +// indistinqushable from printf("%d", long_v) -> +// since printf("%d", int_v) works, printf("%d", long_v) has to work. +//But "clean" solution would be to add "l" to d,i,o,x,X. +//Probably makes sense to go all the way to "ll" then. +//Coreutils support long long-sized arguments. + /* needed - try "printf %" without it */ if (!strchr("diouxXfeEgGcs", *f)) { bb_error_msg("%s: invalid format", direc_start); -- cgit v1.2.3