diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2026-03-22 00:05:47 +0300 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2026-03-23 12:10:59 +0300 |
| commit | 0fdbe845534f4e0b7243b7a44562c4486ce25fb4 (patch) | |
| tree | 5709dc6a40a97488b73191cd8ac0ac32072d0853 | |
| parent | 4520b96b8136ba2465a3f4dc5c3fb8bdf3d92e4e (diff) | |
| download | linux-0fdbe845534f4e0b7243b7a44562c4486ce25fb4.tar.xz | |
fanotify: replace deprecated strcpy in fanotify_info_copy_{name,name2}
strcpy() has been deprecated [1] because it performs no bounds checking
on the destination buffer, which can lead to buffer overflows. Replace
it with the safer strscpy().
Link: https://www.kernel.org/doc/html/latest/process/deprecated.html#strcpy [1]
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20260321210544.519259-4-thorsten.blum@linux.dev
Signed-off-by: Jan Kara <jack@suse.cz>
| -rw-r--r-- | fs/notify/fanotify/fanotify.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/notify/fanotify/fanotify.h b/fs/notify/fanotify/fanotify.h index 39e60218df7c..a0619e7694d5 100644 --- a/fs/notify/fanotify/fanotify.h +++ b/fs/notify/fanotify/fanotify.h @@ -2,6 +2,7 @@ #include <linux/fsnotify_backend.h> #include <linux/path.h> #include <linux/slab.h> +#include <linux/string.h> #include <linux/exportfs.h> #include <linux/hashtable.h> @@ -218,7 +219,7 @@ static inline void fanotify_info_copy_name(struct fanotify_info *info, return; info->name_len = name->len; - strcpy(fanotify_info_name(info), name->name); + strscpy(fanotify_info_name(info), name->name, name->len + 1); } static inline void fanotify_info_copy_name2(struct fanotify_info *info, @@ -228,7 +229,7 @@ static inline void fanotify_info_copy_name2(struct fanotify_info *info, return; info->name2_len = name->len; - strcpy(fanotify_info_name2(info), name->name); + strscpy(fanotify_info_name2(info), name->name, name->len + 1); } /* |