On 2024/11/6 10:58, Daniel Yang wrote:
strcpy is deprecated. Kernel docs recommend replacing strcpy with
strscpy. The function strcpy() return value isn't used so there
shouldn't be an issue replacing with the safer alternative strscpy.
It's better to add "f2fs: " prefix for commit title.
Signed-off-by: Daniel Yang <danielyangk...@gmail.com>
---
fs/f2fs/super.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index 87ab5696b..ee8861b95 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -1158,7 +1158,7 @@ static int parse_options(struct super_block *sb, char
*options, bool is_remount)
break;
}
- strcpy(ext[ext_cnt], name);
+ strscpy(ext[ext_cnt], name);
Needs to handle return value of strscpy()?
F2FS_OPTION(sbi).compress_ext_cnt++;
kfree(name);
break;
@@ -1187,7 +1187,7 @@ static int parse_options(struct super_block *sb, char
*options, bool is_remount)
break;
}
- strcpy(noext[noext_cnt], name);
+ strscpy(noext[noext_cnt], name);
Ditto,
Thanks,
F2FS_OPTION(sbi).nocompress_ext_cnt++;
kfree(name);
break;
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
