diff options
| author |   Eric Biggers <ebiggers@google.com> | 2016-10-11 10:36:12 -0700 |
|---|---|---|
| committer |   Jaegeuk Kim <jaegeuk@kernel.org> | 2016-11-23 12:10:57 -0800 |
| commit | 0c0b471e43e7acf0747c6eb410863bf78c14750d (patch) | |
| tree | 21711c32abd717f6b28ad52c5e17d43b2ec38cc5 /fs/f2fs/node.c | |
| parent | f2fs: fix error handling in fsync_node_pages (diff) | |
| download | linux-dev-0c0b471e43e7acf0747c6eb410863bf78c14750d.tar.xz linux-dev-0c0b471e43e7acf0747c6eb410863bf78c14750d.zip | |
f2fs: fix sparse warnings
f2fs contained a number of endianness conversion bugs.
Also, one function should have been 'static'.
Found with sparse by running 'make C=2 CF=-D__CHECK_ENDIAN__ fs/f2fs/'
Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/node.c')
| -rw-r--r-- | fs/f2fs/node.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c index 975909c146a1..6f99907bc8d7 100644 --- a/fs/f2fs/node.c +++ b/fs/f2fs/node.c @@ -270,8 +270,9 @@ static void cache_nat_entry(struct f2fs_sb_info *sbi, nid_t nid, e = grab_nat_entry(nm_i, nid); node_info_from_raw_nat(&e->ni, ne); } else { - f2fs_bug_on(sbi, nat_get_ino(e) != ne->ino || - nat_get_blkaddr(e) != ne->block_addr || + f2fs_bug_on(sbi, nat_get_ino(e) != le32_to_cpu(ne->ino) || + nat_get_blkaddr(e) != + le32_to_cpu(ne->block_addr) || nat_get_version(e) != ne->version); } } |