On Fri, Nov 8, 2024 at 1:47 AM Chao Yu <c...@kernel.org> wrote: > > 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; >
Ok, I've added the changes and sent a v2 of the patch. Lmk if there's any other issues. - Daniel _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel