summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2010-03-20 11:22:10 +0000
committerChris Mason <chris.mason@oracle.com>2010-03-30 21:19:09 -0400
commit6cf8bfbf5e88edfb09a2bf0631a067060f534592 (patch)
tree70cc8040189bcd623e72ab1f09c6dc4fb40741e5
parentc2b96929e2ca6914cf4a66cd8fe2a34c4a98277f (diff)
downloadlinux-stable-6cf8bfbf5e88edfb09a2bf0631a067060f534592.tar.gz
linux-stable-6cf8bfbf5e88edfb09a2bf0631a067060f534592.tar.bz2
linux-stable-6cf8bfbf5e88edfb09a2bf0631a067060f534592.zip
Btrfs: check btrfs_get_extent return for IS_ERR()
btrfs_get_extent() never returns NULL, only a valid pointer or ERR_PTR() Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Chris Mason <chris.mason@oracle.com>
-rw-r--r--fs/btrfs/ioctl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c
index 74d89133f768..2b7dd88fc54f 100644
--- a/fs/btrfs/ioctl.c
+++ b/fs/btrfs/ioctl.c
@@ -510,7 +510,7 @@ static int should_defrag_range(struct inode *inode, u64 start, u64 len,
em = btrfs_get_extent(inode, NULL, 0, start, len, 0);
unlock_extent(io_tree, start, start + len - 1, GFP_NOFS);
- if (!em)
+ if (IS_ERR(em))
return 0;
}