mm/damon/sysfs-schemes: use strscpy() to instead of strncpy()
authorXu Panda <xu.panda@zte.com.cn>
Mon, 9 Jan 2023 11:46:55 +0000 (19:46 +0800)
committerAndrew Morton <akpm@linux-foundation.org>
Thu, 19 Jan 2023 01:12:59 +0000 (17:12 -0800)
The implementation of strscpy() is more robust and safer.
That's now the recommended way to copy NUL-terminated strings.

Link: https://lkml.kernel.org/r/202301091946553770006@zte.com.cn
Signed-off-by: Xu Panda <xu.panda@zte.com.cn>
Signed-off-by: Yang Yang <yang.yang29@zte.com.cn>
Reviewed-by: SeongJae Park <sj@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/damon/sysfs-schemes.c

index f0dabe3..86edca6 100644 (file)
@@ -353,8 +353,7 @@ static ssize_t memcg_path_store(struct kobject *kobj,
        if (!path)
                return -ENOMEM;
 
-       strncpy(path, buf, count);
-       path[count] = '\0';
+       strscpy(path, buf, count + 1);
        filter->memcg_path = path;
        return count;
 }