diff options
| author | Rusty Russell <rusty@rustcorp.com.au> | 2008-01-29 23:13:20 +0100 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2008-01-29 07:13:21 +0100 |
| commit | efa5345e39d01deef349c120f55ac6b6eabe7457 (patch) | |
| tree | 7c8d0919aa380a0db7ac748d6c4571c00aa5c639 /kernel | |
| parent | module: wait for dependent modules doing init. (diff) | |
| download | linux-efa5345e39d01deef349c120f55ac6b6eabe7457.tar.xz linux-efa5345e39d01deef349c120f55ac6b6eabe7457.zip | |
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')
| -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) mutex_lock(&module_mutex); } /* Store the name of the last unloaded module for diagnostic purposes */ - sprintf(last_unloaded_module, mod->name); + strlcpy(last_unloaded_module, mod->name, sizeof(last_unloaded_module)); free_module(mod); out: |