diff options
| author | Stefan Behrens <sbehrens@giantdisaster.de> | 2012-10-11 07:25:16 -0600 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-25 15:40:03 -0400 |
| commit | 84167d190569eedcdb24bf2499bdda437e442962 (patch) | |
| tree | d577be39043333f60016aa744f4ebcb084eb8e5d /fs | |
| parent | 661bec6ba884b86517ef5ea529aabb281a7198d9 (diff) | |
| download | linux-84167d190569eedcdb24bf2499bdda437e442962.tar.gz linux-84167d190569eedcdb24bf2499bdda437e442962.tar.xz | |
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.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) 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; } |