[U-Boot] [PATCH 1/4] tools: imx8image: check lseek return value

Peng Fan peng.fan at nxp.com
Thu Nov 1 12:15:15 UTC 2018


Hi Fabio,

> -----Original Message-----
> From: Fabio Estevam [mailto:festevam at gmail.com]
> Sent: 2018年11月1日 18:18
> To: Peng Fan <peng.fan at nxp.com>
> Cc: Stefano Babic <sbabic at denx.de>; Fabio Estevam
> <fabio.estevam at nxp.com>; U-Boot-Denx <u-boot at lists.denx.de>; dl-linux-imx
> <linux-imx at nxp.com>
> Subject: Re: [U-Boot] [PATCH 1/4] tools: imx8image: check lseek return value
> Hi Peng,
> 
> On Thu, Nov 1, 2018 at 3:42 AM Peng Fan <peng.fan at nxp.com> wrote:
> 
> >         size = sbuf.st_size;
> > -       lseek(ifd, offset, SEEK_SET);
> > +       if (lseek(ifd, offset, SEEK_SET) == -1) {
> > +               fprintf(stderr, "%s: lseek error %s\n",
> > +                       __func__, strerror(errno));
> > +               exit(EXIT_FAILURE);
> > +       }
> 
> What about something like this instead?
> 
> ret = lseek(ifd, offset, SEEK_SET);
> if (ret < 0) {
>                fprintf(stderr, "%s: lseek error: %d\n", __func__, ret);
>                exit(EXIT_FAILURE);
> }
lseek only returns -1 when error. Is the coding style you proposed is preferred than
what this patch has? 
I saw fit_image use if (xxx < 0) and fw_env.c use if (xxx == -1),
there are also others use ret = lseek().

Thanks,
Peng.


More information about the U-Boot mailing list