Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20211215212126.GK7074@brightrain.aerifal.cx>
Date: Wed, 15 Dec 2021 16:21:26 -0500
From: Rich Felker <dalias@...c.org>
To: Andrew Snyder <arsnyder16@...il.com>
Cc: Quentin Rameau <quinq@...th.space>, musl@...ts.openwall.com
Subject: Re: print does not support variable width plus padding

On Wed, Dec 15, 2021 at 01:37:43PM -0500, Andrew Snyder wrote:
> That is probably true but I think on accident trying to replicate my
> original issue which was using the native function

With the program:

#include <stdio.h>
int main(int argc, char **argv)
{
	printf("%0*i\n", 2, argc);
}

(written using argc so it can't be collapsed to a constant string at
compile time) I get output of "01\n" as expected. If you think there's
a bug in musl for this or related functionality, can you provide a
minimal C test case?

> On Wed, Dec 15, 2021 at 1:09 PM Quentin Rameau <quinq@...th.space> wrote:
> 
> > Hi Andrew,
> >
> > > Sorry accidentally sent before attaching this
> > >
> > > ~# docker run -it --rm alpine /bin/ash
> > > / # /lib/libc.musl-x86_64.so.1
> > > musl libc (x86_64)
> > > Version 1.2.2
> > > Dynamic Program Loader
> > > Usage: /lib/libc.musl-x86_64.so.1 [options] [--] pathname [args]
> > > / # printf %0*i 2 1
> > > ash: %0*i: invalid format
> >
> > This looks like you found a bug in Busybox printf implementation.

FWIW I suspect the problem is that Busybox is not recognizing the 0
character as a flag (which it is, in the printf grammar) and thinks
it's the leading character of a width, making the * specifier for
width invalid (since a width was already seen).

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.