[U-Boot] [PATCH] ext4fs: le32_to_cpu() used on a 16-bit field

Simon Glass sjg at chromium.org
Sun Jul 21 20:13:59 CEST 2013


Hi Andreas,

On Sun, Jul 21, 2013 at 2:53 AM, Andreas Bießmann <
andreas.devel at googlemail.com> wrote:

> From: Rommel Custodio <sessyargc+uboot at gmail.com>
>
> Fix reading ext4_extent_header struture on BE machines.
> Some 16 bit fields where converted to 32 bit fields, due to the byte swap
> on BE
> machines the containing value was corrupted. Therefore reading ext4
> filesystems
> on BE machines where broken before.
>
> Signed-off-by: Rommel Custodio <sessyargc+uboot at gmail.com>
> [sent via git-send-email; rework commit message]
> Signed-off-by: Andreas Bießmann <andreas.devel at googlemail.com>
>

Reviewed-by: Simon Glass <sjg at chromium.org>

I tested it on sandbox with the sandbox block patch (not yet merged). It
worked fine with my simple test (loading a kernel and a 70MB executable)
with and without this patch unfortunately, but anyway:

Tested-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list