[U-Boot] [PATCH v7 02/13] lib: strto: fix metric suffix parsing in strtoul[l]
Miquel Raynal
miquel.raynal at bootlin.com
Mon Sep 3 06:49:43 UTC 2018
Hi Stefan,
Stefan Roese <sr at denx.de> wrote on Sat, 1 Sep 2018 10:48:25 +0200:
> On 31.08.2018 16:57, Miquel Raynal wrote:
> > While 1kB or 1kiB will be parsed correctly, 1k will return the right
> > amount, but the metric suffix will not be escaped once the char
> > pointer updated. Fix this situation by simplifying the move of the
> > endp pointer.
> > > Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> > ---
> > lib/strto.c | 22 ++++++++++------------
> > 1 file changed, 10 insertions(+), 12 deletions(-)
> > > diff --git a/lib/strto.c b/lib/strto.c
> > index 84f8d92d57..502a0153e7 100644
> > --- a/lib/strto.c
> > +++ b/lib/strto.c
> > @@ -97,12 +97,11 @@ unsigned long ustrtoul(const char *cp, char **endp, unsigned int base)
> > case 'K':
> > case 'k':
> > result *= 1024;
> > - if ((*endp)[1] == 'i') {
> > - if ((*endp)[2] == 'B')
> > - (*endp) += 3;
> > - else
> > - (*endp) += 2;
> > - }
> > + (*endp)++;
> > + if (**endp == 'i')
> > + (*endp)++;
> > + if (**endp == 'B')
> > + (*endp)++;
> > }
> > return result;
> > }
> > @@ -122,12 +121,11 @@ unsigned long long ustrtoull(const char *cp, char **endp, unsigned int base)
> > case 'K':
> > case 'k':
> > result *= 1024;
> > - if ((*endp)[1] == 'i') {
> > - if ((*endp)[2] == 'B')
> > - (*endp) += 3;
> > - else
> > - (*endp) += 2;
> > - }
> > + (*endp)++;
> > + if (**endp == 'i')
> > + (*endp)++;
> > + if (**endp == 'B')
> > + (*endp)++;
> > }
> > return result;
> > }
> >
> Even though KiB is not equal to KB in general (at least in Linux
> userspace AFAIK), lets not change this in U-Boot and always use
> KiB and KB as a representation for 1024 (instead of 1000). So:
That's right I did not mentioned it in the commit log. I could
update it to reflect that it is intentional to mix 'k' and 'kiB' as a
representation of '* 12014' (already the case, but being clarified in
the above change).
Thanks,
Miquèl
More information about the U-Boot
mailing list