diff options
| author | Roel Kluin <roel.kluin@gmail.com> | 2009-09-23 18:57:30 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-24 10:21:05 -0400 |
| commit | 3886de938cc23d5208a34064d8363d6d7b54b461 (patch) | |
| tree | 8a01235eb668c9eb0d3ec200fe26bfc7c0834ba1 /fs/adfs | |
| parent | 458e5ff13e1bed050990d97e9aa55bcdafc951a7 (diff) | |
adfs: remove redundant test on unsigned
unsigned block cannot be less than 0.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/adfs')
| -rw-r--r-- | fs/adfs/inode.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/fs/adfs/inode.c b/fs/adfs/inode.c index 798cb071d132..3f57ce4bee5d 100644 --- a/fs/adfs/inode.c +++ b/fs/adfs/inode.c | |||
| @@ -19,9 +19,6 @@ static int | |||
| 19 | adfs_get_block(struct inode *inode, sector_t block, struct buffer_head *bh, | 19 | adfs_get_block(struct inode *inode, sector_t block, struct buffer_head *bh, |
| 20 | int create) | 20 | int create) |
| 21 | { | 21 | { |
| 22 | if (block < 0) | ||
| 23 | goto abort_negative; | ||
| 24 | |||
| 25 | if (!create) { | 22 | if (!create) { |
| 26 | if (block >= inode->i_blocks) | 23 | if (block >= inode->i_blocks) |
| 27 | goto abort_toobig; | 24 | goto abort_toobig; |
| @@ -34,10 +31,6 @@ adfs_get_block(struct inode *inode, sector_t block, struct buffer_head *bh, | |||
| 34 | /* don't support allocation of blocks yet */ | 31 | /* don't support allocation of blocks yet */ |
| 35 | return -EIO; | 32 | return -EIO; |
| 36 | 33 | ||
| 37 | abort_negative: | ||
| 38 | adfs_error(inode->i_sb, "block %d < 0", block); | ||
| 39 | return -EIO; | ||
| 40 | |||
| 41 | abort_toobig: | 34 | abort_toobig: |
| 42 | return 0; | 35 | return 0; |
| 43 | } | 36 | } |
