Re: [PATCH] kexec_file: Adjust type of kexec_purgatory

From: Kees Cook
Date: Wed May 10 2017 - 15:54:11 EST


On Tue, May 9, 2017 at 5:15 PM, Eric W. Biederman <ebiederm@xxxxxxxxxxxx> wrote:
> Kees Cook <keescook@xxxxxxxxxxxx> writes:
>> kernel/kexec_file.c:33:13: warning: array âkexec_purgatoryâ assumed to
>> have one element
>> char __weak kexec_purgatory[];
>> ^~~~~~~~~~~~~~~
>
> Nor does "void *kexec_purgatory" as that says at the address known as
> kexec_purgatory is a void pointer not a blob a bytes that can be used
> for something interesting.
>
> Better to get rid of the __weak and deal with that fallout.

Agreed. This is actually pretty easy, since only x86 and PPC use
kexec_file, and both define purgatories. I'll send an updated patch.

Thanks!

-Kees

--
Kees Cook
Pixel Security