diff options
author | Brian Foster <bfoster@redhat.com> | 2024-04-25 12:44:00 +0200 |
---|---|---|
committer | Miklos Szeredi <mszeredi@redhat.com> | 2024-05-08 09:31:21 +0200 |
commit | 96d88f65adfbcaca153afd7d3e20d74ba379c599 (patch) | |
tree | df08d2b2b8e6c8091bf4ca9ab02a45f7f4242ddf /fs | |
parent | fuse: verify zero padding in fuse_backing_map (diff) | |
download | linux-96d88f65adfbcaca153afd7d3e20d74ba379c599.tar.xz linux-96d88f65adfbcaca153afd7d3e20d74ba379c599.zip |
virtiofs: include a newline in sysfs tag
The internal tag string doesn't contain a newline. Append one when
emitting the tag via sysfs.
[Stefan] Orthogonal to the newline issue, sysfs_emit(buf, "%s", fs->tag) is
needed to prevent format string injection.
Signed-off-by: Brian Foster <bfoster@redhat.com>
Fixes: a8f62f50b4e4 ("virtiofs: export filesystem tags through sysfs")
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/fuse/virtio_fs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/fuse/virtio_fs.c b/fs/fuse/virtio_fs.c index 322af827a232..bb3e941b9503 100644 --- a/fs/fuse/virtio_fs.c +++ b/fs/fuse/virtio_fs.c @@ -170,7 +170,7 @@ static ssize_t tag_show(struct kobject *kobj, { struct virtio_fs *fs = container_of(kobj, struct virtio_fs, kobj); - return sysfs_emit(buf, fs->tag); + return sysfs_emit(buf, "%s\n", fs->tag); } static struct kobj_attribute virtio_fs_tag_attr = __ATTR_RO(tag); |