summaryrefslogtreecommitdiffstats
path: root/fs/internal.h
diff options
context:
space:
mode:
authorZach Brown <zab@redhat.com>2013-01-08 19:37:58 +0000
committerJosef Bacik <josef@toxicpanda.com>2013-01-14 13:53:23 -0500
commitf276795627045a3c599a60b476767861e4318c7d (patch)
tree8e69e69fed84ae5b7ed675d490180e01c5a71f64 /fs/internal.h
parentf9e4fb53938de5db01950c9dfe479703b2f5c964 (diff)
downloadblackbird-obmc-linux-f276795627045a3c599a60b476767861e4318c7d.tar.gz
blackbird-obmc-linux-f276795627045a3c599a60b476767861e4318c7d.zip
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>
Diffstat (limited to 'fs/internal.h')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud