diff options
| author |   Thorsten Blum <thorsten.blum@linux.dev> | 2025-03-08 20:46:32 +0100 |
|---|---|---|
| committer |   Petr Pavlu <petr.pavlu@suse.com> | 2025-03-10 11:56:26 +0100 |
| commit | 085c5e37427d22381efa6e8a660dd810e8ef36ef (patch) | |
| tree | 9bd59ddbc96136999d448311de343cba2f62f8a4 /kernel/module/main.c | |
| parent | module: Replace deprecated strncpy() with strscpy() (diff) | |
| download | wireguard-linux-085c5e37427d22381efa6e8a660dd810e8ef36ef.tar.xz wireguard-linux-085c5e37427d22381efa6e8a660dd810e8ef36ef.zip | |
module: Remove unnecessary size argument when calling strscpy()
The size parameter is optional and strscpy() automatically determines
the length of the destination buffer using sizeof() if the argument is
omitted. This makes the explicit sizeof() unnecessary. Remove it to
shorten and simplify the code.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://lore.kernel.org/r/20250308194631.191670-2-thorsten.blum@linux.dev
Signed-off-by: Petr Pavlu <petr.pavlu@suse.com>
Diffstat (limited to 'kernel/module/main.c')
| -rw-r--r-- | kernel/module/main.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/module/main.c b/kernel/module/main.c index e1ed5d82f45f..9195849e13c4 100644 --- a/kernel/module/main.c +++ b/kernel/module/main.c @@ -787,8 +787,8 @@ SYSCALL_DEFINE2(delete_module, const char __user *, name_user, async_synchronize_full(); /* Store the name and taints of the last unloaded module for diagnostic purposes */ - strscpy(last_unloaded_module.name, mod->name, sizeof(last_unloaded_module.name)); - strscpy(last_unloaded_module.taints, module_flags(mod, buf, false), sizeof(last_unloaded_module.taints)); + strscpy(last_unloaded_module.name, mod->name); + strscpy(last_unloaded_module.taints, module_flags(mod, buf, false)); free_module(mod); /* someone could wait for the module in add_unformed_module() */ |