diff options
author | Stefan Behrens <sbehrens@giantdisaster.de> | 2012-10-11 09:25:16 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-25 15:40:03 -0400 |
commit | 84167d190569eedcdb24bf2499bdda437e442962 (patch) | |
tree | d577be39043333f60016aa744f4ebcb084eb8e5d /fs/btrfs/extent_io.c | |
parent | 661bec6ba884b86517ef5ea529aabb281a7198d9 (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.c | 4 |
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; |
4106 | err: | 4106 | err: |
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 | } |