ubi-utils updates for gcc-4.3.x
Artem Bityutskiy
dedekind at infradead.org
Mon Dec 8 08:09:42 EST 2008
On Mon, 2008-12-08 at 13:00 +0200, Artem Bityutskiy wrote:
> >From man scanf:
>
> RETURN VALUE
> These functions return the number of input items successfully
> matched and assigned, which can be fewer than provided for, or even zero
> in the event of an early matching failure.
>
> The value EOF is returned if the end of input is reached before either
> the first successful conversion or a matching failure occurs. EOF is
> also returned if a read error occurs, in which case the error indicator
> for the stream (see ferror(3)) is set, and errno is set indicate the
> error.
>
> On this basis, I would say you should test for EOF, not for 0.
>
> > + if (!scanf("%3s", buf))
> > + return 0;
>
> Similarly.
>
If you are fine with this, I may amend your patch and push it, because
otherwise it looks OK for me.
--
Best regards,
Artem Bityutskiy (Битюцкий Артём)
More information about the linux-mtd
mailing list