diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2008-01-29 17:13:20 -0500 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2008-01-29 01:13:21 -0500 |
commit | efa5345e39d01deef349c120f55ac6b6eabe7457 (patch) | |
tree | 7c8d0919aa380a0db7ac748d6c4571c00aa5c639 /kernel/module.c | |
parent | c9a3ba55bb5da03fc7d707709a7fe078fe1aa0a0 (diff) |
module: Fix gratuitous sprintf in module.c
Andrew sent an older version of this patch: we shouldn't use sprintf
to copy a string.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'kernel/module.c')
-rw-r--r-- | kernel/module.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/module.c b/kernel/module.c index 17314691d3cc..276abd7b7ff7 100644 --- a/kernel/module.c +++ b/kernel/module.c | |||
@@ -738,7 +738,7 @@ sys_delete_module(const char __user *name_user, unsigned int flags) | |||
738 | mutex_lock(&module_mutex); | 738 | mutex_lock(&module_mutex); |
739 | } | 739 | } |
740 | /* Store the name of the last unloaded module for diagnostic purposes */ | 740 | /* Store the name of the last unloaded module for diagnostic purposes */ |
741 | sprintf(last_unloaded_module, mod->name); | 741 | strlcpy(last_unloaded_module, mod->name, sizeof(last_unloaded_module)); |
742 | free_module(mod); | 742 | free_module(mod); |
743 | 743 | ||
744 | out: | 744 | out: |