]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
module: Remove unnecessary size argument when calling strscpy()
authorThorsten Blum <thorsten.blum@linux.dev>
Sat, 8 Mar 2025 19:46:32 +0000 (20:46 +0100)
committerPetr Pavlu <petr.pavlu@suse.com>
Mon, 10 Mar 2025 10:56:26 +0000 (11:56 +0100)
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>
kernel/module/main.c

index e1ed5d82f45feb8dabfa4bbdbbc90e8d9c15368d..9195849e13c4d88bdbf9b163bffcafd0191024c3 100644 (file)
@@ -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() */