• Hans de Goede's avatar
    efi/libstub/random: Initialize pointer variables to zero for mixed mode · 818c7ce7
    Hans de Goede authored
    Commit:
    
      0d959814 ("x86: efi/random: Invoke EFI_RNG_PROTOCOL to seed the UEFI RNG table")
    
    causes the drivers/efi/libstub/random.c code to get used on x86 for the first time.
    
    But this code was not written with EFI mixed mode in mind (running a 64
    bit kernel on 32 bit EFI firmware), this causes the kernel to crash during
    early boot when running in mixed mode.
    
    The problem is that in mixed mode pointers are 64 bit, but when running on
    a 32 bit firmware, EFI calls which return a pointer value by reference only
    fill the lower 32 bits of the passed pointer, leaving the upper 32 bits
    uninitialized which leads to crashes.
    
    This commit fixes this by initializing pointers which are passed by
    reference to EFI calls to NULL before passing them, so that the upper 32
    bits are initialized to 0.
    Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
    Signed-off-by: default avatarArd Biesheuvel <ardb@kernel.org>
    Cc: Arvind Sankar <nivedita@alum.mit.edu>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: linux-efi@vger.kernel.org
    Fixes: 0d959814 ("x86: efi/random: Invoke EFI_RNG_PROTOCOL to seed the UEFI RNG table")
    Link: https://lkml.kernel.org/r/20191224132909.102540-3-ardb@kernel.orgSigned-off-by: default avatarIngo Molnar <mingo@kernel.org>
    818c7ce7
random.c 5.59 KB