diff options
| author | Xu Panda <xu.panda@zte.com.cn> | 2023-01-09 19:46:55 +0800 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2023-01-18 17:12:59 -0800 |
| commit | b6f00c9190c8e694c9b2b38e7cc63964f7a99195 (patch) | |
| tree | 3ddcfcd3da7aba63e34bef40bd266296b583024a | |
| parent | e8dfc854eef20ac7663996f61837299887f380fc (diff) | |
| download | lwn-b6f00c9190c8e694c9b2b38e7cc63964f7a99195.tar.gz lwn-b6f00c9190c8e694c9b2b38e7cc63964f7a99195.zip | |
mm/damon/sysfs-schemes: use strscpy() to instead of strncpy()
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>
| -rw-r--r-- | mm/damon/sysfs-schemes.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mm/damon/sysfs-schemes.c b/mm/damon/sysfs-schemes.c index f0dabe3e2dc0..86edca66aab1 100644 --- a/mm/damon/sysfs-schemes.c +++ b/mm/damon/sysfs-schemes.c @@ -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; } |