summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorStefan Behrens <sbehrens@giantdisaster.de>2012-10-11 07:25:16 -0600
committerChris Mason <chris.mason@fusionio.com>2012-10-25 15:40:03 -0400
commit84167d190569eedcdb24bf2499bdda437e442962 (patch)
treed577be39043333f60016aa744f4ebcb084eb8e5d /fs
parent661bec6ba884b86517ef5ea529aabb281a7198d9 (diff)
downloadlinux-84167d190569eedcdb24bf2499bdda437e442962.tar.gz
linux-84167d190569eedcdb24bf2499bdda437e442962.tar.bz2
linux-84167d190569eedcdb24bf2499bdda437e442962.zip
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')
-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)
return eb;
err:
- for (i--; i >= 0; i--)
- __free_page(eb->pages[i]);
+ for (; i > 0; i--)
+ __free_page(eb->pages[i - 1]);
__free_extent_buffer(eb);
return NULL;
}