Re: [PATCH] xendump: fix failure to match arm/aarch64 elf format of xendump file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----Original Message-----
> From: Goodbach <goodbach@xxxxxxxxx>
> Date: Wed, 12 Aug 2020 11:22:29 +0800
> 
> Resolves: https://github.com/crash-utility/crash/pull/61
> Signed-off-by: Goodbach <goodbach@xxxxxxxxx>

I'm not sure whether it's needed to also add the ARM one to 32-bit side,
but the patch itself looks good to me.

Acked-by: Kazuhito Hagio <k-hagio-ab@xxxxxxx>

Thanks,
Kazu

> ---
>  xendump.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/xendump.c b/xendump.c
> index 70cf261..a81817d 100644
> --- a/xendump.c
> +++ b/xendump.c
> @@ -2055,6 +2055,16 @@ xc_core_elf_verify(char *file, char *buf)
>  				goto bailout;
>  			break;
> 
> +		case EM_ARM:
> +			if (machine_type_mismatch(file, "ARM", NULL, 0))
> +				goto bailout;
> +			break;
> +
> +		case EM_AARCH64:
> +			if (machine_type_mismatch(file, "ARM64", NULL, 0))
> +				goto bailout;
> +			break;
> +
>  		default:
>  			if (machine_type_mismatch(file, "(unknown)", NULL, 0))
>  				goto bailout;
> 
> 
> --
> Crash-utility mailing list
> Crash-utility@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/crash-utility


--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux