summaryrefslogtreecommitdiffstats
path: root/fs/btrfs/extent-tree.c
diff options
context:
space:
mode:
authorMiao Xie <miaox@cn.fujitsu.com>2013-02-07 06:02:44 +0000
committerJosef Bacik <jbacik@fusionio.com>2013-02-20 12:59:41 -0500
commit8696c53304f16fde9368b9d5c89a5acb4a815d4c (patch)
tree856777de858d660b2d1c8c38facd5197407ba668 /fs/btrfs/extent-tree.c
parent2b8195bb5717729e4e94ab4ad73a543feaafb0a2 (diff)
downloadtalos-op-linux-8696c53304f16fde9368b9d5c89a5acb4a815d4c.tar.gz
talos-op-linux-8696c53304f16fde9368b9d5c89a5acb4a815d4c.zip
Btrfs: fix memory leak of pending_snapshot->inherit
The argument "inherit" of btrfs_ioctl_snap_create_transid() was assigned to NULL during we created the snapshots, so we didn't free it though we called kfree() in the caller. But since we are sure the snapshot creation is done after the function - btrfs_ioctl_snap_create_transid() - completes, it is safe that we don't assign the pointer "inherit" to NULL, and just free it in the caller of btrfs_ioctl_snap_create_transid(). In this way, the code can become more readable. Reported-by: Alex Lyakas <alex.btrfs@zadarastorage.com> Cc: Arne Jansen <sensille@gmx.net> Signed-off-by: Miao Xie <miaox@cn.fujitsu.com> Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs/btrfs/extent-tree.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud