btrfs: abort unlink trans in missed error case
authorZach Brown <zab@redhat.com>
Tue, 2 Apr 2013 21:02:16 +0000 (21:02 +0000)
committerJosef Bacik <jbacik@fusionio.com>
Mon, 6 May 2013 19:54:36 +0000 (15:54 -0400)
commitd4e3991b9945906528c7abb627d759ea43f53bce
tree8ef759bd9b3348528c29bb0433c57eae921d6ad2
parentf63e0cca9174dd8b47380f61565ab7b921d6ac2c
btrfs: abort unlink trans in missed error case

__btrfs_unlink_inode() aborts its transaction when it sees errors after
it removes the directory item.  But it missed the case where
btrfs_del_dir_entries_in_log() returns an error.  If this happens then
the unlink appears to fail but the items have been removed without
updating the directory size.  The directory then has leaked bytes in
i_size and can never be removed.

Adding the missing transaction abort at least makes this failure
consistent with the other failure cases.

I noticed this while reading the code after someone on irc reported
having a directory with i_size but no entries.  I tested it by forcing
btrfs_del_dir_entries_in_log() to return -ENOMEM.

Signed-off-by: Zach Brown <zab@redhat.com>
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
fs/btrfs/inode.c