[U-Boot] [PATCH v7 01/13] lib: strto: parse all lowercase metric prefixes in ustrtoul[l]
Miquel Raynal
miquel.raynal at bootlin.com
Mon Sep 3 06:47:39 UTC 2018
Hi Stefan,
Stefan Roese <sr at denx.de> wrote on Sat, 1 Sep 2018 10:43:01 +0200:
> On 31.08.2018 16:57, Miquel Raynal wrote:
> > Both ustrtoul and ustrtoull interpret 1k but not 1m or 1g. Even if the
> > SI symbols for Mega and Giga are 'M' and 'G', certain entries of
> > eg. mtdparts also use (wrongly) the metric prefix 'm' and 'g'.
> > > I do not see how parsing lowercase prefixes could break anything, so
> > parse them like their uppercase counterpart.
> > > Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> > ---
> > lib/strto.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> > > diff --git a/lib/strto.c b/lib/strto.c
> > index 7f6076909a..84f8d92d57 100644
> > --- a/lib/strto.c
> > +++ b/lib/strto.c
> > @@ -87,9 +87,11 @@ unsigned long ustrtoul(const char *cp, char **endp, unsigned int base)
> > unsigned long result = simple_strtoul(cp, endp, base);
> > switch (**endp) {
> > case 'G':
> > + case 'g':
> > result *= 1024;
> > /* fall through */
> > case 'M':
> > + case 'm':
> > result *= 1024;
> > /* fall through */
> > case 'K':
> > @@ -110,9 +112,11 @@ unsigned long long ustrtoull(const char *cp, char **endp, unsigned int base)
> > unsigned long long result = simple_strtoull(cp, endp, base);
> > switch (**endp) {
> > case 'G':
> > + case 'g':
> > result *= 1024;
> > /* fall through */
> > case 'M':
> > + case 'm':
>
> Wouldn't it be better, to use tolower() on the char and drop all the
> upper case checks completely - also for the 'K' case?
Sure, I can do that!
Thanks,
Miquèl
More information about the U-Boot
mailing list