diff options
author | Chen Gang <gang.chen@asianux.com> | 2013-05-29 21:18:43 -0400 |
---|---|---|
committer | Helge Deller <deller@gmx.de> | 2013-06-01 08:29:01 -0400 |
commit | ea99b1adf22abd62bdcf14b1c9a0a4d3664eefd8 (patch) | |
tree | a5ae4d443b53c2f338e78bf4799937a2d4501f1e /arch/parisc/kernel | |
parent | 766039022a480ede847659daaa78772bdcc598ae (diff) |
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 76b63e726a53..1e95b2000ce8 100644 --- a/arch/parisc/kernel/setup.c +++ b/arch/parisc/kernel/setup.c | |||
@@ -69,7 +69,8 @@ void __init setup_cmdline(char **cmdline_p) | |||
69 | /* called from hpux boot loader */ | 69 | /* called from hpux boot loader */ |
70 | boot_command_line[0] = '\0'; | 70 | boot_command_line[0] = '\0'; |
71 | } else { | 71 | } else { |
72 | strcpy(boot_command_line, (char *)__va(boot_args[1])); | 72 | strlcpy(boot_command_line, (char *)__va(boot_args[1]), |
73 | COMMAND_LINE_SIZE); | ||
73 | 74 | ||
74 | #ifdef CONFIG_BLK_DEV_INITRD | 75 | #ifdef CONFIG_BLK_DEV_INITRD |
75 | if (boot_args[2] != 0) /* did palo pass us a ramdisk? */ | 76 | if (boot_args[2] != 0) /* did palo pass us a ramdisk? */ |