This should be rejected as in Coreutils:
$ printf %
./printf: %: invalid conversion specification
The problem is that printf does not validate the format specifier. One
possible fix would be to add a check along these lines this after line 201 in
printf.c:
direc_start = f++;
+ if (*f == '\0')
+ fprintf(stderr, "invalid conversion specification");
direc_length = 1;
field_width = precision = -1;
if (*f == '%') {
bb_putchar('%');
break;
}
Thanks,
Cristian
printf.diff [^] (2,511 bytes) 07-17-08 02:15 
