diff options
| author | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2012-11-28 16:12:41 +0900 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2012-12-11 13:43:42 +0900 |
| commit | 25ca923b2a766b9c93b63777ead351137533a623 (patch) | |
| tree | 6414cd58a9573177310a9f0ccfdfaca9cf45eb70 /fs/f2fs/hash.c | |
| parent | cf0e3a64cad19acd5904946d0647d751c1671620 (diff) | |
| download | op-kernel-dev-25ca923b2a766b9c93b63777ead351137533a623.zip op-kernel-dev-25ca923b2a766b9c93b63777ead351137533a623.tar.gz | |
f2fs: fix endian conversion bugs reported by sparse
This patch should resolve the bugs reported by the sparse tool.
Initial reports were written by "kbuild test robot" managed by fengguang.wu.
In my local machines, I've tested also by running:
> make C=2 CF="-D__CHECK_ENDIAN__"
Accordingly, I've found lots of warnings and bugs related to the endian
conversion. And I've fixed all at this moment.
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs/hash.c')
| -rw-r--r-- | fs/f2fs/hash.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/f2fs/hash.c b/fs/f2fs/hash.c index 098a196..beb155e 100644 --- a/fs/f2fs/hash.c +++ b/fs/f2fs/hash.c @@ -92,7 +92,6 @@ f2fs_hash_t f2fs_dentry_hash(const char *name, int len) hash = buf[0]; minor_hash = buf[1]; - f2fs_hash = hash; - f2fs_hash &= ~F2FS_HASH_COL_BIT; + f2fs_hash = cpu_to_le32(hash & ~F2FS_HASH_COL_BIT); return f2fs_hash; } |
