[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/8] linux/arm: fix ARM Linux header layout
From: |
Daniel Kiper |
Subject: |
Re: [PATCH v2 1/8] linux/arm: fix ARM Linux header layout |
Date: |
Thu, 11 Mar 2021 17:18:12 +0100 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Sun, Oct 25, 2020 at 02:49:34PM +0100, Ard Biesheuvel wrote:
> The hdr_offset member of the ARM Linux image header appears at
> offset 0x3c, matching the PE/COFF spec's placement of the COFF
> header offset in the MS-DOS header. We're currently off by four,
> so fix that.
>
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
I was asked by Dimitri to take this patch. I double checked it. It
looks that it impacts only one grub_file_read() and one grub_dprintf().
So, there is no or minimal risk at this point and no big advantage
for upstream. Though, due to low risk, I will take it for downstream
advantage.
So, Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
Until I hear any objections in an hour or so...
Daniel
> ---
> include/grub/arm/linux.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/grub/arm/linux.h b/include/grub/arm/linux.h
> index 2e98a6689696..bcd5a7eb186e 100644
> --- a/include/grub/arm/linux.h
> +++ b/include/grub/arm/linux.h
> @@ -30,7 +30,7 @@ struct linux_arm_kernel_header {
> grub_uint32_t magic;
> grub_uint32_t start; /* _start */
> grub_uint32_t end; /* _edata */
> - grub_uint32_t reserved2[4];
> + grub_uint32_t reserved2[3];
> grub_uint32_t hdr_offset;
> };
>
> --
> 2.17.1