diff options
author | Chen Gang <gang.chen@asianux.com> | 2013-05-30 01:18:43 (GMT) |
---|---|---|
committer | Helge Deller <deller@gmx.de> | 2013-06-01 12:29:01 (GMT) |
commit | ea99b1adf22abd62bdcf14b1c9a0a4d3664eefd8 (patch) | |
tree | a5ae4d443b53c2f338e78bf4799937a2d4501f1e /arch/parisc/kernel | |
parent | 766039022a480ede847659daaa78772bdcc598ae (diff) | |
download | linux-fsl-qoriq-ea99b1adf22abd62bdcf14b1c9a0a4d3664eefd8.tar.xz |
parisc: kernel: using strlcpy() instead of strcpy()
'boot_args' is an input args, and 'boot_command_line' has a fix length.
So use strlcpy() instead of strcpy() to avoid memory overflow.
Signed-off-by: Chen Gang <gang.chen@asianux.com>
Acked-by: Kyle McMartin <kyle@mcmartin.ca>
Signed-off-by: Helge Deller <deller@gmx.de>
Diffstat (limited to 'arch/parisc/kernel')
-rw-r--r-- | arch/parisc/kernel/setup.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/parisc/kernel/setup.c b/arch/parisc/kernel/setup.c index 76b63e7..1e95b20 100644 --- a/arch/parisc/kernel/setup.c +++ b/arch/parisc/kernel/setup.c @@ -69,7 +69,8 @@ void __init setup_cmdline(char **cmdline_p) /* called from hpux boot loader */ boot_command_line[0] = '\0'; } else { - strcpy(boot_command_line, (char *)__va(boot_args[1])); + strlcpy(boot_command_line, (char *)__va(boot_args[1]), + COMMAND_LINE_SIZE); #ifdef CONFIG_BLK_DEV_INITRD if (boot_args[2] != 0) /* did palo pass us a ramdisk? */ |