diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-09-25 07:39:17 -0400 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-09-30 18:34:56 -0400 |
| commit | 8a21984d5d86540ba32b4b346d8fe6426018b7a7 (patch) | |
| tree | f74aef0bfae8cd51e3efea3d2c003ab84a71699b /fs/f2fs | |
| parent | 44c16156512f33c81e382a1e1df9524e26a7026a (diff) | |
f2fs: potential shift wrapping buf in f2fs_trim_fs()
My static checker complains that segment is a u64 but only the lower 31
bits can be used before we hit a shift wrapping bug.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/segment.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h index a7e3e9112100..afb73627a8ec 100644 --- a/fs/f2fs/segment.h +++ b/fs/f2fs/segment.h | |||
| @@ -55,7 +55,7 @@ | |||
| 55 | #define TOTAL_BLKS(sbi) (TOTAL_SEGS(sbi) << sbi->log_blocks_per_seg) | 55 | #define TOTAL_BLKS(sbi) (TOTAL_SEGS(sbi) << sbi->log_blocks_per_seg) |
| 56 | 56 | ||
| 57 | #define MAX_BLKADDR(sbi) (SEG0_BLKADDR(sbi) + TOTAL_BLKS(sbi)) | 57 | #define MAX_BLKADDR(sbi) (SEG0_BLKADDR(sbi) + TOTAL_BLKS(sbi)) |
| 58 | #define SEGMENT_SIZE(sbi) (1 << (sbi->log_blocksize + \ | 58 | #define SEGMENT_SIZE(sbi) (1ULL << (sbi->log_blocksize + \ |
| 59 | sbi->log_blocks_per_seg)) | 59 | sbi->log_blocks_per_seg)) |
| 60 | 60 | ||
| 61 | #define START_BLOCK(sbi, segno) (SEG0_BLKADDR(sbi) + \ | 61 | #define START_BLOCK(sbi, segno) (SEG0_BLKADDR(sbi) + \ |