Pass the right arguments to EFI_PXE_BASE_CODE_TFTP_READ_FILE

A wrong pointer was being passed to EFI_PXE_BASE_CODE_TFTP_READ_FILE,
preventing us from getting the file size back from the tftp call, ensuring
that we don't have enough information to properly secureboot-validate the
retrieved image.

Add MokListRT option rom entry.

Signed-off-by: Peter Jones <pjones@redhat.com>

Update TODO with missing description.

Signed-off-by: Peter Jones <pjones@redhat.com>

Made TODO represent the present.

Signed-off-by: Peter Jones <pjones@redhat.com>

Make SHIM_LOCK_GUID a first-class object with a symbol.

Right now the CA is checking if shim builds expose a particular version
of the shim protocol.  To do this, they're looking for SHIM_LOCK_GUID's
value in the resulting binary.

Currently, with SHIM_LOCK_GUID as a macro that gets assigned to local
variables, that means they have to compensate for mov instructions mixed
in with the actual value.  This is completely absurd, so promote it to a
first-class object with a symbol to make it both easy to find and
continuous.

Signed-off-by: Peter Jones <pjones@redhat.com>

Don't print things on the screen by default when everything works.

There's no point to this text, and it generally confuses people.

Signed-off-by: Peter Jones <pjones@redhat.com>

Don't print that fallback isn't found in should_use_fallback()

The call can simply fail if it isn't found - which will be the case on
removeable install media.

Signed-off-by: Peter Jones <pjones@redhat.com>

Fix some pointer casting issues.

This also fixes the size of an empty vendor_cert or dbx_cert.

Signed-off-by: Peter Jones <shim-owner@fedoraproject.org>

Bump version to 0.4

Since I've finally merged in the "sections" branch, best to increment
the version number.

Signed-off-by: Peter Jones <pjones@redhat.com>

Make DBX be included in build if the environment is set right.

Signed-off-by: Peter Jones <pjones@redhat.com>

Make .vendor_cert get the right flags set.

Signed-off-by: Peter Jones <pjones@redhat.com>

add a .gitignore

Signed-off-by: Peter Jones <pjones@redhat.com>

Move embedded certificates to their own section.

With this change, the embedded certificate and dbx lists (vendor_cert,
vendor_cert_size, vendor_dbx, and vendor_dbx_size) wind up being in a
section named .vendor_cert, and so will look something like:
------
fenchurch:~/devel/github.com/shim$ objdump -h shim.efi

shim.efi:     file format pei-x86-64

Sections:
Idx Name          Size      VMA               LMA               File off  Algn
  0 .eh_frame     000174a8  0000000000005000  0000000000005000  00000400  2**3
                  CONTENTS, ALLOC, LOAD, READONLY, DATA
  1 .text         000aa7e1  000000000001d000  000000000001d000  00017a00  2**4
                  CONTENTS, ALLOC, LOAD, READONLY, CODE
  2 .reloc        0000000a  00000000000c8000  00000000000c8000  000c2200  2**0
                  CONTENTS, ALLOC, LOAD, READONLY, DATA
  3 .data         00031228  00000000000c9000  00000000000c9000  000c2400  2**5
                  CONTENTS, ALLOC, LOAD, DATA
  4 .vendor_cert  00000375  00000000000fb000  00000000000fb000  000f3800  2**0
                  CONTENTS, READONLY
  5 .dynamic      000000f0  00000000000fc000  00000000000fc000  000f3c00  2**3
                  CONTENTS, ALLOC, LOAD, DATA
  6 .rela         0002afa8  00000000000fd000  00000000000fd000  000f3e00  2**3
                  CONTENTS, ALLOC, LOAD, READONLY, DATA
  7 .dynsym       0000f1f8  0000000000128000  0000000000128000  0011ee00  2**3
                  CONTENTS, ALLOC, LOAD, READONLY, DATA
------

This simplifies a security audit, because it means that different
versions of shim with substantially the same code with different keys
will be more easily comperable, and therefore logic differences may be
more easily identified.

This also means that if there's a trusted build you want to use, you can
remove the certificates, implant new ones, and have it signed, and the
code sections won't change.

Signed-off-by: Peter Jones <pjones@redhat.com>

vendor_cert_size's size in the binary should be 4, not -4.

The thing about subtraction is that the minuend needs to be before the
subtrahend in the text.

Signed-off-by: Peter Jones <pjones@redhat.com>

Remove FALLBACK_OBJS during clean as well.

Signed-off-by: Peter Jones <pjones@redhat.com>

Make sure all the Makefiles use the same arguments for mmx/sse/ms_abi.

Signed-off-by: Peter Jones <pjones@redhat.com>

EFI_PXE_BASE_CODE_DHCPV6_PACKET is in gnu-efi-3.0t

Don't use MMX and SSE registers, they aren't initialized.

GCC 4.8.0 will try to use these by default, and you'll wind up looping
across the (uninitialized!) trap handler for uninitialized instructions.

Signed-off-by: Peter Jones <pjones@redhat.com>

Bump version to 0.3

Signed-off-by: Peter Jones <pjones@redhat.com>

Use MS ABI instead of terrible wrappers.

This means that we now require gnu-efi 3.0s

Signed-off-by: Peter Jones <pjones@redhat.com>

Use the correct define on Open.

The value here doesn't actually change any, but we should still use the
right name.

Signed-off-by: Peter Jones <pjones@redhat.com>

Add some error messages when things don't work.

Signed-off-by: Peter Jones <pjones@redhat.com>

Remove some unnecessary code.

Signed-off-by: Peter Jones <pjones@redhat.com>

Fix some minor type errors.

Signed-off-by: Peter Jones <pjones@redhat.com>

Pass parameters correctly when booting.

Signed-off-by: Peter Jones <pjones@redhat.com>

[fallback] Try to execute the first new boot option.

I'm told rebooting is sometimes unreliable when called here, and we'll
get bootx64.efi loaded anyway.  I'll just assume that's true and try to
load the first option, since it's clearly what we'd prefer happens next.

Signed-off-by: Peter Jones <pjones@redhat.com>

Explain byte order handling better.

Signed-off-by: Peter Jones <pjones@redhat.com>

Don't update BootOrder until all csv files are processed

Reset the system after restoring the boot entries

Fix crash due to memory allocation

Get rid of extra "continue".

It's confusing, and it doesn't actually accomplish anything when applied
to *either* loop.

Signed-off-by: Peter Jones <pjones@redhat.com>

Fix error checking on AllocateZeroPool() in update_boot_order()

Signed-off-by: Peter Jones <pjones@redhat.com>

Make shim use fallback when appropriate.

If we're called as /BOOT/EFI/BOOT*.EFI, and /BOOT/EFI/FALLBACK.EFI exists,
give it a shot.

Signed-off-by: Peter Jones <pjones@redhat.com>

Add a fallback loader for when shim is invoked as BOOTX64.EFI

If shim is invoked as \EFI\BOOT\BOOT*.EFI and a file exists named
\EFI\BOOT\FALLBACK.EFI, try it instead of our second stage.  So don't
put fallback.efi on your install media in \EFI\BOOT, because that won't
do whatever it is you're hoping for, unless you're hoping not to start
the installer.

So here's the process for using this:
in /EFI/fedora/ (or whichever directory you happen to own), you put:
  shim.efi
  grub.efi
  boot.csv - format is: shim.efi,Nice Label,cmdline arguments,comments
           - filenames refer only to files in this directory, with no
     leading characters such as L"./" or L"/EFI/fedora/"
           - note that while this is CSV, the character encoding is
     UCS-2

and if /EFI/BOOT/BOOTX64.EFI doesn't already exist, then in /EFI/BOOT:
  shim.efi as BOOTX64.EFI
  fallback.efi

Signed-off-by: Peter Jones <pjones@redhat.com>

Add StrCSpn()

Signed-off-by: Peter Jones <pjones@redhat.com>

Adopt the UEFI shell style LoadOptions

The previous commit, 14d4b8e, caused shim failed to parse the name
of the 2nd stage loader in UEFI shell. Amend parsing of the name the
2nd stage loader to be compatible with UEFI shell.

To create an boot entry for elilo.efi:

 # efibootmgr -c -L "shim elilo" -l "efi\\shim.efi" -u "shim.efi elilo.efi"

Get rid of extra label.

Signed-off-by: Peter Jones <pjones@redhat.com>

Don't put the directory in the file path twice.

Sometimes when we're creating paths, the ImagePath can contain the
directory name already.  If that happens, don't add it in again.

Signed-off-by: Peter Jones <pjones@redhat.com>

UEFI Shell sticks the UCS2 of li->FilePath in li->LoadOptions. Ignore it.

If li->LoadOptions tells us to execute our own binary, it's clearly not
what we want to do for the second stage.  So simply ignore that case.

Signed-off-by: Peter Jones <pjones@redhat.com>

Make sure the menu shows when the callback fails

Since Pause() doesn't clear the key from the input queue, the next
ReadKeyStroke reads the queued key instead of the new one. If the
user presses "Enter", MokManager exits directly without showing
the menu again.

Add support for deleting specific keys

MOK doesn't include the signature owner

Add a general function for password matching

Reboot the system after enrolling/erasing keys

Clear the screen before erasing keys

Get the second stage loader from the Load Options

This commit replaces the 2nd stage loader path with the first
argument in the Load Options and moves the rest arguments (if any)
to the Load Options for the 2nd stage loader.

For example, to make shim to load elilo.efi, just create a new
boot entry with efibootmgr:

 # efibootmgr -c -L "shim elilo" -l "efi\\shim.efi" -u "elilo.efi"

Remove debug code

secure_mode() was altered to always return true for debug purposes, and this
accidentally got committed to mainline. Fix that.

Sign MokManager with a locally-generated key

shim needs to verify that MokManager hasn't been modified, but we want to
be able to support configurations where shim is shipped without a vendor
certificate. This patch adds support for generating a certificate at build
time, incorporating the public half into shim and signing MokManager with
the private half. It uses pesign and nss, but still requires openssl for
key generation. Anyone using sbsign will need to figure this out for
themselves.

Don't fail if there's no network devices

findNetboot() would continue blindly even if no PXE-capable devices were
found. Fix that.

Bump version

This seems pretty much functionally complete, so let's call it 0.2.

Merge branch 'mok' into netboot

Conflicts:
shim.c

Add comments

Merge branch 'mok' into netboot

Conflicts:
Makefile
shim.c

Fix AuthenticodeVerify loop

Cert needs to be modified inside the Index loop, not outside it. This is unlikely to
ever trigger since there will typically only be one X509 certificate per
EFI_SIGNATURE_LIST, but fix it anyway.

Fix signature checking

We could potentially find a valid signature and then fail to validate it
due to not breaking out of the outer while loop.

Fix double free

load_image() didn't allocate PathName, don't have it free it.

Fix up some types

Type-checking the UEFI calls picked up a couple of problems. Fix them up.

Add documentation of the Mok variables

Brief overview of the function and format of the various variables used
by Shim and MokManager.

Merge branch 'mok' of github.com:mjg59/shim into mok

Check the vendor blacklist correctly

Initialize the size of vendor dbx as 0

The size of vendor dbx must be 0 if there is no vendor dbx provided
or the functions of db check will crash.

Clean up password setting

Permit clearing of the password, and avoid a case where choosing not to set
a password would result in an error message on exit. Fix the same problem
with MokSB.

Improve signature validation enable/disable

The logic used in checking the signature validation password was a bit
ugly. Improve that so it behaves rather more as expected.

Boot unsigned binaries if we're not in secure mode

read_header would fail if the binary was unsigned, even if we weren't then
going to verify the signature. Move that check to the verify function
instead.

Miscellaneous small fixups

Fixes for some small bugs noticed during review

Add another missing screen clearing

Another case where we were drawing text over existing text.

Merge branch 'mok' of github.com:mjg59/shim into mok

Fix password hash calculation

This was hardcoded, rather than being based on the actual password length,
resulting in incorrect hashes being generated.

Update image validation enable/disable

Update this to match the new mokutil behaviour

Delete MokList properly

A cut and paste error meant that attempts to delete MokList were instead
appending a zero-length addition.

Clean up checks for MokManager entry

Add a helper function and tidy up the calls for getting into MokManager

Fix key database parsing

The pointer to the certificate needs to be incremented by the size of the
entire certificate, not just the certificate data.

Support a vendor-specific DBX list.

In some rare corner cases, it's useful to add a blacklist of things that
were allowed by a copy of shim that was never signed by the UEFI signing
service.  In these cases it's okay for them to go into a local dbx,
rather than taking up precious flash.

Signed-off-by: Peter Jones <pjones@redhat.com>

Clear screen before prompting

We were drawing prompts on top of existing text, which was less than
ideal.

Don't print SHA1 sum when calculating file fingerprints

There's no point in printing the SHA1 of a SHA256...

Clean up timeout counter handling

Reduce menu redrawing by only redrawing the invalidated section of the menu
during the timeout countdown.

Add MOK password auth

Add support for setting an MOK password. The OS passes down a password hash.
MokManager then presents an option for setting a password. Selecting it
prompts the user for the same password again. If they match, the hash is
enrolled into a boot services variable and MokManager will prompt for the
password whenever it's started.

Pause on callback failures

If a callback returns any kind of failure, wait for a keypress in order to
give the user an opportunity to read any failure messages.

Skip signature checking if insecure

If we're configured to run untrusted code, print a message and skip the
validation checks.

Add support for disabling signature verification

Provide a mechanism for a physically present end user to disable signature
verification. This is handled by the OS passing down a variable that
contains a UINT32 and a SHA256 hash. If this variable is present, MokManager
prompts the user to choose whether to enable or disable signature validation
(depending on the value of the UINT32). They are then asked to type the
passphrase that matches the hash. This then saves a boot services variable
which is checked by shim, and if set will skip verification of signatures.

Add section headers

Provide a little more contextual information when people are in shim
menus.

Add draft version of Neil's netboot code

Reallocate the DevPath space for the volume label

The size of the DevPath string array was not sufficient to append
the volume label. This patch extends the size for the label and
re-enables the menu freeing.

Remove LoadImage/StartImage support

Some systems will show an error dialog if LoadImage() returned
EFI_ACCESS_DENIED, which then requires physical user interaction to skip.
Let's just remove the LoadImage/StartImage code, since the built-in code
is theoretically equivalent.

Switch to using db format for MokList and MokNew

Using the same format as the UEFI key databases makes it easier for the
kernel to parse and extract keys from MOK, and also permits MOK to contain
multiple key or hash types. Additionally, add support for enrolling hashes.

Split out hashing

We want to be able to generate hashes, so split out the hash generation
function from the verification function

Add missing header define

Add SHA1 support

In theory vendors could blacklist binaries with SHA1, so make sure we
calculate and check that hash as well.

Add menu header

Add a basic header to the menu to make it clearer what's going on.
Define SHIM_VENDOR in order to override the default.

Clear screen on exit

Add timeout support

We should time out if there aren't any keypresses at the top level menu
within a reasonable timeframe.

Fix whitespace

Remove unnecessary newline

The newline character shouldn't be present in the menu item string

Pass the size of MokNew to the key enrollment function

The size of MokNew was missing and it caused crash when enrolling new MOKs
with mokutil

Callbacks should return INTN, not UINTN

Remove unused variable

Don't lose the last file in top-level directories

The filesystem callback was failing to account for the additional menu
item to return to the filesystem list, and so the last file entry in the
root directory would be missing from the list.

Fix filesystem enrollment

We would always enrol a single key, deleting any existing keys in the
database. Fix that up.

Add an auth argument to store_keys()

If the user is manually installing keys from a filesystem then we don't need
to ask for the key password.

Add __attribute__ ((packed)) to MokListNode definition

Packing this lets us use MokListNode with the binary MokList representation.