btrfs: fix btrfs_cont_expand() freeing IS_ERR em
authorZach Brown <zab@redhat.com>
Tue, 8 Jan 2013 19:37:58 +0000 (19:37 +0000)
committerJosef Bacik <josef@toxicpanda.com>
Mon, 14 Jan 2013 18:53:23 +0000 (13:53 -0500)
commitf276795627045a3c599a60b476767861e4318c7d
tree8e69e69fed84ae5b7ed675d490180e01c5a71f64
parentf9e4fb53938de5db01950c9dfe479703b2f5c964
btrfs: fix btrfs_cont_expand() freeing IS_ERR em

btrfs_cont_expand() tries to free an IS_ERR em as it gets an error from
btrfs_get_extent() and breaks out of its loop.

An instance of -EEXIST was reported in the wild:

  https://bugzilla.redhat.com/show_bug.cgi?id=874407

I have no idea if that -EEXIST is surprising, or not.  Regardless, this
error handling should be cleaned up to handle other reasonable errors
(ENOMEM, EIO; whatever).

This seemed to be the only buggy freeing of the relatively rare IS_ERR
em so I opted to fix the caller rather than teach free_extent_map() to
use IS_ERR_OR_NULL().

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