[U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy

qiang.zhao at freescale.com qiang.zhao at freescale.com
Wed May 6 03:40:48 CEST 2015


On Tue, May 5, 2015 at 5:41 PM, Michael Trimarchi wrote:

> -----Original Message-----
> From: Michael Trimarchi [mailto:michael at amarulasolutions.com]
> Sent: Tuesday, May 05, 2015 5:41 PM
> To: Zhao Qiang-B45475
> Cc: u-boot at lists.denx.de; Sun York-R58495
> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy
> 
> Hi
> 
> On Tue, May 5, 2015 at 5:15 PM, qiang.zhao at freescale.com
> <qiang.zhao at freescale.com> wrote:
> > On Tue, May 5, 2015 at 4:33 PM, Michael Trimarchi wrote:
> >> -----Original Message-----
> >> From: Michael Trimarchi [mailto:michael at amarulasolutions.com]
> >> Sent: Tuesday, May 05, 2015 4:33 PM
> >> To: Zhao Qiang-B45475
> >> Cc: u-boot at lists.denx.de; Sun York-R58495
> >> Subject: Re: [U-Boot] [PATCH 2/2] qe: use strncpy instead of strcpy
> >>
> >> Hi
> >>
> >> On Tue, May 5, 2015 at 3:53 PM, Zhao Qiang <B45475 at freescale.com>
> wrote:
> >> > strncpy is safer than strcpy, use it to instead of strcpy.
> >> >
> >> > Signed-off-by: Zhao Qiang <B45475 at freescale.com>
> >> > ---
> >> >  drivers/qe/qe.c | 2 +-
> >> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >> >
> >> > diff --git a/drivers/qe/qe.c b/drivers/qe/qe.c index
> >> > f1df0a4..08620b2
> >> > 100644
> >> > --- a/drivers/qe/qe.c
> >> > +++ b/drivers/qe/qe.c
> >> > @@ -435,7 +435,7 @@ int qe_upload_firmware(const struct qe_firmware
> >> *firmware)
> >> >          * saved microcode information and put in the new.
> >> >          */
> >> >         memset(&qe_firmware_info, 0, sizeof(qe_firmware_info));
> >> > -       strcpy(qe_firmware_info.id, (char *)firmware->id);
> >> > +       strncpy(qe_firmware_info.id, (char *)firmware->id, 62);
> >>
> >> is 62 somenthing that can be calculated?
> >
> > The fireware->id is defined 62 bytes in the struct.
> >
> 
> and you don't like sizeof

Thank you for your suggestion.
If there are something else need to be modify,
I will use sizeof instead.

> 
> Michael
> 
> 
> 
> >
> >>
> >> >         qe_firmware_info.extended_modes = firmware->extended_modes;
> >> >         memcpy(qe_firmware_info.vtraps, firmware->vtraps,
> >> >                 sizeof(firmware->vtraps));
> >> > --
> >> > 2.1.0.27.g96db324
> >> >
> >> > _______________________________________________
> >> > U-Boot mailing list
> >> > U-Boot at lists.denx.de
> >> > http://lists.denx.de/mailman/listinfo/u-boot
> > Best Regards
> > Zhao Qiang
> 
> 
> 
> --
> | Michael Nazzareno Trimarchi                     Amarula Solutions BV |
> | COO  -  Founder                                      Cruquiuskade 47 |
> | +31(0)851119172                                 Amsterdam 1018 AM NL |
> |                  [`as] http://www.amarulasolutions.com               |


More information about the U-Boot mailing list