diff options
| author | Daniel Rosenberg <drosen@google.com> | 2017-06-26 18:21:56 -0700 |
|---|---|---|
| committer | Daniel Rosenberg <drosen@google.com> | 2017-06-27 17:05:21 -0700 |
| commit | 16759392d588c6042e4496943ba718ef403e9735 (patch) | |
| tree | c7db80617eac97b33f1a74e7d922c87471ea6183 /fs | |
| parent | 575a44c7cc5e526ff0311eb66d5c6cb2d931406c (diff) | |
ANDROID: squashfs: Fix endianness issue
Code in squashfs_process_blocks was not correctly assigning
length. Casting to u16* introduced endianness issues on some
architectures.
Signed-off-by: Daniel Rosenberg <drosen@google.com>
Bug: 35257858
Change-Id: I9efaef4bc531b7469de79cf94738ade2dd6e6a8c
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/squashfs/block.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/fs/squashfs/block.c b/fs/squashfs/block.c index 4e3e0863f5ea..b3b95e2ae2ff 100644 --- a/fs/squashfs/block.c +++ b/fs/squashfs/block.c @@ -121,11 +121,12 @@ static void squashfs_process_blocks(struct squashfs_read_request *req) if (req->data_processing == SQUASHFS_METADATA) { /* Extract the length of the metadata block */ - if (req->offset != msblk->devblksize - 1) - length = *((u16 *)(bh[0]->b_data + req->offset)); - else { - length = bh[0]->b_data[req->offset]; - length |= bh[1]->b_data[0] << 8; + if (req->offset != msblk->devblksize - 1) { + length = le16_to_cpup((__le16 *) + (bh[0]->b_data + req->offset)); + } else { + length = (unsigned char)bh[0]->b_data[req->offset]; + length |= (unsigned char)bh[1]->b_data[0] << 8; } req->compressed = SQUASHFS_COMPRESSED(length); req->data_processing = req->compressed ? SQUASHFS_DECOMPRESS |