aboutsummaryrefslogtreecommitdiffstats
path: root/fs/btrfs/extent_io.c
diff options
context:
space:
mode:
authorStefan Behrens <sbehrens@giantdisaster.de>2012-10-11 09:25:16 -0400
committerChris Mason <chris.mason@fusionio.com>2012-10-25 15:40:03 -0400
commit84167d190569eedcdb24bf2499bdda437e442962 (patch)
treed577be39043333f60016aa744f4ebcb084eb8e5d /fs/btrfs/extent_io.c
parent661bec6ba884b86517ef5ea529aabb281a7198d9 (diff)
Btrfs: Fix wrong error handling code
gcc says "warning: comparison of unsigned expression >= 0 is always true" because i is an unsigned long. And gcc is right this time. Signed-off-by: Stefan Behrens <sbehrens@giantdisaster.de>
Diffstat (limited to 'fs/btrfs/extent_io.c')
-rw-r--r--fs/btrfs/extent_io.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
index 64dc93f64bc0..a32ebfeb91cf 100644
--- a/fs/btrfs/extent_io.c
+++ b/fs/btrfs/extent_io.c
@@ -4104,8 +4104,8 @@ struct extent_buffer *alloc_dummy_extent_buffer(u64 start, unsigned long len)
4104 4104
4105 return eb; 4105 return eb;
4106err: 4106err:
4107 for (i--; i >= 0; i--) 4107 for (; i > 0; i--)
4108 __free_page(eb->pages[i]); 4108 __free_page(eb->pages[i - 1]);
4109 __free_extent_buffer(eb); 4109 __free_extent_buffer(eb);
4110 return NULL; 4110 return NULL;
4111} 4111}