Roel Kluin <[EMAIL PROTECTED]> wrote: > Roel Kluin wrote: > > This patch was not yet tested. Please confirm it's right. > > was too quick with the send button. the batch below is probably better > --- > strncpy does not append '\0' if the length of the source string equals > the size parameter, strlcpy does. > > Signed-off-by: Roel Kluin <[EMAIL PROTECTED]>
Acked-by: Kou Ishizaki <[EMAIL PROTECTED]> I tested Roel's 2nd patch and it works good on Celleb. > --- > diff --git a/arch/powerpc/platforms/celleb/setup.c > b/arch/powerpc/platforms/celleb/setup.c > index f27ae1e..cbe09d9 100644 > --- a/arch/powerpc/platforms/celleb/setup.c > +++ b/arch/powerpc/platforms/celleb/setup.c > @@ -81,8 +81,7 @@ static void celleb_show_cpuinfo(struct seq_file *m) > > static int __init celleb_machine_type_hack(char *ptr) > { > - strncpy(celleb_machine_type, ptr, sizeof(celleb_machine_type)); > - celleb_machine_type[sizeof(celleb_machine_type)-1] = 0; > + strlcpy(celleb_machine_type, ptr, sizeof(celleb_machine_type)); > return 0; > } > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@ozlabs.org > https://ozlabs.org/mailman/listinfo/linuxppc-dev _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev