diff options
author | Xiongfeng Wang <xiongfeng.wang@linaro.org> | 2018-01-08 19:51:22 +0800 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2018-01-22 16:08:21 +0100 |
commit | 6670d4c2d9b7d352cbf90cd18c516bb6444acf34 (patch) | |
tree | 774ef2a1c3266b19e4be07829686c4f2129bae90 /fs | |
parent | 6f794e3c5c8f8fdd3b5bb20d9ded894e685b5bbe (diff) | |
download | op-kernel-dev-6670d4c2d9b7d352cbf90cd18c516bb6444acf34.zip op-kernel-dev-6670d4c2d9b7d352cbf90cd18c516bb6444acf34.tar.gz |
btrfs: use correct string length in DEV_INFO ioctl
gcc-8 reports:
fs/btrfs/ioctl.c: In function 'btrfs_ioctl':
./include/linux/string.h:245:9: warning: '__builtin_strncpy' specified
bound 1024 equals destination size [-Wstringop-truncation]
We need one less byte or call strlcpy() to make it a nul-terminated
string. This is done on the next line anyway, but we want to avoid the
warning.
Signed-off-by: Xiongfeng Wang <xiongfeng.wang@linaro.org>
Reviewed-by: David Sterba <dsterba@suse.com>
[ update changelog ]
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/btrfs/ioctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index f6d4d58..f573cad 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -2798,7 +2798,7 @@ static long btrfs_ioctl_dev_info(struct btrfs_fs_info *fs_info, struct rcu_string *name; name = rcu_dereference(dev->name); - strncpy(di_args->path, name->str, sizeof(di_args->path)); + strncpy(di_args->path, name->str, sizeof(di_args->path) - 1); di_args->path[sizeof(di_args->path) - 1] = 0; } else { di_args->path[0] = '\0'; |