diff options
| author | Stefan Behrens <sbehrens@giantdisaster.de> | 2012-10-11 15:25:16 +0200 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-25 21:40:03 +0200 |
| commit | 84167d190569eedcdb24bf2499bdda437e442962 (patch) | |
| tree | d577be39043333f60016aa744f4ebcb084eb8e5d /fs | |
| parent | Fix a sign bug causing invalid memory access in the ino_paths ioctl. (diff) | |
| download | linux-84167d190569eedcdb24bf2499bdda437e442962.tar.xz 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.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; } |