sd-boot: stub: Obtain PE section offsets from RAM, not disk (#6250)
authorMatthew Garrett <mjg59@google.com>
Fri, 30 Jun 2017 18:27:47 +0000 (11:27 -0700)
committerLennart Poettering <lennart@poettering.net>
Mon, 3 Jul 2017 16:14:09 +0000 (18:14 +0200)
commitd4cbada2a95667c4d5d4310298bfcb446b1357b5
treec7915ded4a0afb62475fabe36341226e589fb6a2
parent9db307820e6f545665fc87f255af737228b7183c
sd-boot: stub: Obtain PE section offsets from RAM, not disk (#6250)

In a Secure Boot scenario the stub loader will have been validated
before execution. A malicious drive could then change the data returned
in future reads, resulting in the loader obtaining incorrect section
offsets and (for instance) allowing the command line to be modified.
Pull that information out of the in-RAM representation of the loader
instead in order to avoid this.

Fixes: #6230

(Lennart did some minor coding style fixes, and renamed pefile.c → pe.c,
as suggested by Kay, given that the file now contains a function whose
name doesn't match the filename as prefix anymore.)
src/boot/efi/boot.c
src/boot/efi/meson.build
src/boot/efi/pe.c [moved from src/boot/efi/pefile.c with 60% similarity]
src/boot/efi/pe.h [moved from src/boot/efi/pefile.h with 67% similarity]
src/boot/efi/stub.c