diff options
| author | Waiman Long <longman@redhat.com> | 2020-06-16 11:31:59 -0400 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2020-06-16 19:24:03 +0200 |
| commit | b091f7fede97cc64f7aaad3eeb37965aebee3082 (patch) | |
| tree | c73c266d76a14da8d3c1c04fad1473797054adca /fs | |
| parent | 5dbb75ed6900048e146247b6325742d92c892548 (diff) | |
| download | linux-b091f7fede97cc64f7aaad3eeb37965aebee3082.tar.bz2 | |
btrfs: use kfree() in btrfs_ioctl_get_subvol_info()
In btrfs_ioctl_get_subvol_info(), there is a classic case where kzalloc()
was incorrectly paired with kzfree(). According to David Sterba, there
isn't any sensitive information in the subvol_info that needs to be
cleared before freeing. So kzfree() isn't really needed, use kfree()
instead.
Signed-off-by: Waiman Long <longman@redhat.com>
Reviewed-by: David Sterba <dsterba@suse.com>
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 168deb8ef68a..e8f7c5f00894 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -2692,7 +2692,7 @@ out: btrfs_put_root(root); out_free: btrfs_free_path(path); - kzfree(subvol_info); + kfree(subvol_info); return ret; } |