f2fs: Fix type of section block count variables
authorShin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Wed, 19 Aug 2020 01:34:48 +0000 (10:34 +0900)
committerJaegeuk Kim <jaegeuk@kernel.org>
Wed, 9 Sep 2020 03:31:33 +0000 (20:31 -0700)
commit123aaf774f70ba48fb9ab064b6ce75a9b64d7b85
tree2e44c6a19c7af918f74464b51dd26830dd7c7955
parent34d4ddd359dbcdf6c5fb3f85a179243d7a1cb7f8
f2fs: Fix type of section block count variables

Commit da52f8ade40b ("f2fs: get the right gc victim section when section
has several segments") added code to count blocks of each section using
variables with type 'unsigned short', which has 2 bytes size in many
systems. However, the counts can be larger than the 2 bytes range and
type conversion results in wrong values. Especially when the f2fs
sections have blocks as many as USHRT_MAX + 1, the count is handled as 0.
This triggers eternal loop in init_dirty_segmap() at mount system call.
Fix this by changing the type of the variables to block_t.

Fixes: da52f8ade40b ("f2fs: get the right gc victim section when section has several segments")
Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/segment.c