Re: why does sscanf() does not interpret number length attributes?

Kay Sievers (lkml001@vrfy.org)
Thu, 3 Jul 2003 00:26:28 +0200


On Wed, Jul 02, 2003 at 04:55:44PM -0400, Richard B. Johnson wrote:
> On Wed, 2 Jul 2003, Kay Sievers wrote:
>
> > I needed a conversion from hex-string to integer and found
> > this mail from Linus suggesting sscanf:
> >
> > http://marc.theaimsgroup.com/?l=linux-kernel&m=101414195507893&w=2
> >
> > but sscanf in linux-2.5/lib/vsprintf.c interpretes length attributes
> > only when the type is a string. It uses simple_strtoul() and it will
> > read the buffer until it finds a non-(hex)digit.
> >
> > int i;
> > char str[] ="34AFFE45XYZ";
> > sscanf(str, "%1x", &i);
> >
> > i will be '0x34AFFE45' instead of the expected '3'.
> >
> > Is this behaviour intended or is just nobody caring about?
> >
>
> The in-kernel vsprintf() is very primative, used for very simple
> things. Note that it doesn't even have "%f". You should just
> do something like:
>
> i = (int) *str + '0';

And what about hex lowercases ?
Do you meant "- '0'" :-)

>
> ... if you need to read part of a number.
>
> You don't really wany to increase the size of the permanent
> in-kernel stuff if you can help. You add any increased functionality
> to your modules so it is used only when needed.

Sure, but the length attribute is already available in a var at processing
and all what is needed is one extra parameter for escaping from the char read loop.

i wouldn't call this "increase of permanent stuff".

thanks
Kay
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/