Android: inital porting of efibootmgr

The libefivar porting is also required.

make_linux_load_option(): check data_size correctly.

Coverity noticed that we're not checking data_size for NULLness before
dereferencing it to pass the value to efi_loadopt_create().  So now we
pass a temporary variable and assign it only if it's non-NULL.

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

unparse_path: remove unneeded include

This fixes an invalid redefinition of wchar_t when compiling with hardened gcc-musl on gentoo.

Bump version to 15

Make conditional default loader path use EFIDIR like fwupdate does.

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

Make default '--loader' build-time configurable.

Each distribution uses a specific 'vendor'-directory in the
'efi' directory on the EFI System Partition to store their
EFI loader.  There's little use in hardcoding the value
for just one in 'efibootmgr', which is displayed in '--help'
and used as default without '--loader'.

Simply use
  make OS_VENDOR=<vendor> EFI_LOADER=<loader>.efi
to get the desired value.  :)

Signed-off-by: Raymund Will <rw@suse.com>

efibootmgr: sanitize `set/get_mirror()`

get_mirror()
- don't short-circuit the assignment of values in case of version mismatch
  as `show_mirror()` happily tries to use them nevertheless
- move redundant warning (from `show_mirror()`s PoV) to `set_mirror()`
- add missing `free(data)`

set_mirror()
- skip obsolete second assignment of `data`
- avoid potentially "uninitialized" access to `above/below4g` by protecting
  it with `opts.set_mirror_hi/_lo` respectively, thus simplifying the code
  down the line
Note: this will prevent unnecessary write-operations!

Signed-off-by: Raymund Will <rw@suse.com>

Add support for parsing optional data as ucs2

Remove extra const keywords gcc 7 gripes about.

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

Don't use -fshort-wchar when building (#63)

It's not needed and is causing build failures with gcc 6. Closes
Debian bug #849651

Signed-off-by: Steve McIntyre <steve@einval.com>

Bump version to 14

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

efibootdump: Get rid of extra debug print

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

Move -flto from cflags to CFLAGS

LTO is an optimization that isn't always desired. This change allows it
to be disabled easily.

Don't bother with --default-symver since we've got no library code here.

ld.gold breaks with it, for basically no reason, but we're not really
using it anyway.

Resolves github issue #56.

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

Make compiler detection slightly better.

Resolves github issue #57.

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

Fix EFIBOOTMGR_VERSION setting and -V output

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

efibootmgr: fixes in man page synopsis and README

- more --delete-bootnum description fixes
- make -r | -y bold
- delete -H and -U (removed in 83f34d39299)

Install efibootdump with make

Create README.md

Fix a typo in popt options.

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

efibootmgr: remove "(hex)" from description of --delete-bootnum

Boot entry selected with --bootnum

efibootmgr 13

- Add support for --sysprep and --driver to support UEFI System Prep
  Applications and UEFI Drivers.
- use efivar's error reporting facility, and show error traces when
  "-v -v" is used.
- Still yet better error codes returned on failures.
- Add -m and -M to support Memory Address Range Mirroring.
- Add efibootdump, to examine Boot* variables found in tarballs in bug
  reports and similar.
- miscellaneous bugfixes.

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

efibootdump: give it the error.h treatment

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

Add support for --sysprep and --driver

Maybe this fixes issue rhinstaller/efibootmgr#40 .

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

.spec: fix requirements for newer efivar and man page path.

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

efibootmgr: use efivar's error facility.

Stop preceeding our version number with a "0.".

It's useless and it just makes people whine about it not being >1.0.

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

README: update github link

README: fix netboot command line since we have acpi bits now.

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

efibootmgr: fix usage of efi_loadopt_path() /again/.

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

efibootmgr: fix up a lot of error result codes.

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

Handle "-@ -" correctly again.

This should fix issue #39.

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

Add lacking description of man for -m and -M option

Currently there's no description of efibootmgr man page
for -m and -M option.

This patch adds the lacking description for -m and -M
option.

Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com>

Add lacking description of man for -m and -M option

Currently there's no description of efibootmgr man page
for -m and -M option.

This patch adds the lacking description for -m and -M
option.

Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com>

Explicitly pass a mode to efi_set_variable() in all cases.

We'll have to do this for efivar-0.24 .

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

efibootmgr.spec: allow building on arm

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

efibootmgr.spec: require popt-devel

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

efibootmgr: fix some types the compiler doesn't like.

Basically uint8_t * vs char *.

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

efibootdump: add efibootdump

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

efibootnext: add make targets (but not default)

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

efoconman: remove from default make targets for now

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

efibootmgr: update for efivar 0.24

As of 0.24 efi_loadopt_pathlen() and efi_loadopt_path() require a limit
argument to avoid overruns of the input data.

This patch adds reasonable limits to those calls.

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

Add -Wextra, because it finds a lot of bugs in GCC6

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

eficonman: intialize poptOptions with named initializers.

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

Add efibootnext beginnings

efibootnext will (eventually) attempt to find any suitable boot device
and set it as BootNext.  To be used when other variable writes have
failed, in case it will use less space.

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

Add eficonman beginnings

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

Fix insusufficient validation check of -M option

The acceptable range of -M (--mirror-above-4G) option should
be from 0 to 50. So the negative value should be rejected.
Otherwise unintended value can be specified.

e.g.

 > efibootmgr -M -3
...
MirroredPercentageAbove4G: 25.00
MirrorMemoryBelow4GB: true
RequestMirroredPercentageAbove4G: 652.36
RequestMirrorMemoryBelow4GB: true

 652.36 is invalid value for MirroredPercentageAbove4G and should
 not be specified.

Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com>

Add LOCALEDIR make goo

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

Simplify directory structure.

We're making a lot less stuff than we did before libefivar.so existed,
and there's not a lot of reason for it.  Collapse the directory
structure entirely, and move to more modern makefiles.

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

Set pkg-config binary as variable

Allow the location of the pkg-config binary to be set using the
parameter PKG_CONFIG. Needed to support cross compilation toolchains.

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

Make it a little clearer what's going on when efi_del_variable() fails.

Right now if efi_del_variable() fails we try a couple of things for
compatibility reasons and we have too many different error messages.
Hopefully this makes it clearer.

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

INSTALL: add note about missing efivar.h error

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

Set guid when reading boot entries in read_vars.

set_active_state use boot->guid, so set it to EFI_GLOBAL_GUID.

Delete BootOrder only when necessary.

Signed-off-by: Shih-Yuan Lee (FourDollars) <sylee@ubuntu.com>

Add -m and -M options to support address range mirror

BIOS and OS pass messages for address range mirror using a pair of
UEFI variables:

   MirrorCurrent: Written by BIOS to tell the OS that the platform supports
address range mirror and to say whether mirroring was set up using
the desired parameters
   MirrorRequest: Written by OS to tell BIOS new paramters to use on next boot

This is documented in the RAS validation guide for memory address range
mirroring for the Xeon E7-v3 family:

  https://software.intel.com/sites/default/files/managed/43/6a/Memory%20Address%20Range%20Mirroring%20Validation%20Guide.pdf

Signed-off-by: Tony Luck <tony.luck@intel.com>

Update version to 0.12

- use -lefivar and -lefiboot to make device paths and load options.

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

Update build deps in the spec to match what libraries we actually use.

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

Make the release procedure a bit more streamlined.

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

Add some language about efivar in INSTALL

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

Get rid of pci-utils and zlib deps; we don't need them any more.

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

Make loadopt function names use efivar-0.19 names.

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

Get us back to where we were before with network entries.

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

We don't need the ucs2/utf8 code either.

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

Get rid of efichar.h; it's no longer used.

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

Fix the github url once more.

This currently lives under the rhinstaller github account.

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

Get rid of manually specified ACPI arguments; we don't need them.

Since we have pretty good detection of acpi hid/uid now, we don't
actually need them to be manually specified anywhere.

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

Make -e 1 and -e 3 work again after the libefiboot shuffle.

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

Make file boot option creation use libefiboot to create the device path.

(right now network options are horribly broken, but they never really
worked anyway, so... yeah, I'll fix them in a subsequent commit.)

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

Add "efibootmgr -C"; create only without appending to BootOrder

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

Get rid of a lot of DP stuff that efivar does now.

Get run of unparse_path() and unparse_raw()

We don't use these any more.

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

Make show_boot_vars use efivars DP formatting code.

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

Don't depend on vendor-defined calloc(n, 0) behavior.

If we don't have any boot variables, we know that 0 is free, so just
return that.

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

Test a couple of pointers for NULL-ness.

In both of these cases buf is NULL if and only if the size is also 0,
and thus we'll fail the test anyway, but coverify isn't noticing that.
And I'm tired of talking to a piece of software about its deficiencies.

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

Free some stuff in error pathways.

This is just to make covscan gripe less.  These variables are legitimate
leaks, though the program is just going to exit afterward anyway.

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

Don't malloc(0).

This is covscan noticing that we're starting at 0 hunks instead of 1
hunk, and that means the first allocation is of 0 bytes, which is inane.

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

Fix buffer overflow when remove_from_boot_order removes nothing

Deleting a boot entry via "-b xxxx -B" also attempts to remove
that entry from boot order via a call to remove_from_boot_order.
Although unusual, it's possible that the entry being deleted is
not in boot order. Correct the handling of this case in
remove_from_boot_order, which malloc's space for the new boot
order list wrongly assuming that at least one entry will be
removed. However, if no entry is removed, then 2 bytes are
overwritten beyond the malloc'ed space. This can result in heap
corruption and possible termination via a SIGABRT if the
corruption is detected by the heap allocation routines.

While there, simplify the routine to do the removal of boot
entries in place in the original data buffer, skip the
unnecessary BootOrder variable update if nothing got removed,
and free the malloc'ed boot_order struct on the way out.

Resolves: RH BZ 1168019

Signed-off-by: Lenny Szubowicz <lszubowi@redhat.com>

Skip unreadable variables, but don't exit when we do so.

Old versions of efibootmgr and libefivar0 allowed the installation of
necessary EFI boot entries on the affected systems. It seems that an old
libefivar0 did not report read errors.

Then libefivar0 started to crash on unreadable variables (see Debian bug
764386). Since that bug has been fixed, efibootmgr doesn't crash
anymore, but the read errors are reported and efibootmgr always fails
with an input/output error.

Resolves: debian bug 768880

Signed-off-by: Jan Echternach <jan@goneko.de>
Signed-off-by: Daniel Jared Dominguez <Jared_Dominguez@Dell.com>
Signed-off-by: Peter Jones <pjones@redhat.com>

Make IPv6 network paths print correctly.

This should fix issue #13 .

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

Make IPv4() device paths print correctly.

We need to give the size to snprintf, not the offset.  The offset is
zero, so it'll only ever tell us the buffer size we need, even if we
have a reasonable buffer.

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

Make the optional_data_len size signed.

Steve McIntyre's testing on i386 with efivars using the legacy path
(i.e. not efivarfs) also found a bug here - we're checking
optional_data_len for negative error values, but it's unsigned.  We're
not going to have 4GB of optional loader data, so just make it signed.

This should fix Issue #21 .

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

Fix MAC() device path printing.

67491a308 accidentally made it print MAC(MAC( .

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

Print a better message if BootOrder is unset.

This resolves issue #7 .

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

Don't name something efi_variable_t; that namespace is used by efivar.

Newer efivar has efi_variable_t as a type for its import/export api
(which efibootmgr can use at some point), so don't make a type with that
name internally here.

This fixes github issue #20

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

Update version to 0.10

- Fix "-n" and friends not being assigned/checked right sometimes from
  0.10.0-1
- Generate more archives to avoid people using github's, because
  they're just bad.

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

Do even more absurd stuff to avoid github's terrible tarballs.

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

Actually /set/ the values we're using to the results of parsing the number.

This should fix https://github.com/vathpela/efibootmgr/issues/18

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

Don't check if '-n' is a current boot entry so early.

We haven't made the list yet, so it'll always be wrong.

This should fix part of https://github.com/vathpela/efibootmgr/issues/18

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

Audit error checking in main().

Previously, one return value wound up not being checked at all,
resulting in https://github.com/vathpela/efibootmgr/issues/14 , and
several weren't checked as they should have been.

Sorry about that.

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

Make the version 0.10.0

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

Make all the other places we're parsing XXXX also do a better job.

This is related to https://github.com/vathpela/efibootmgr/issues/12 .

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

Make -o's behavior actually match the documented behavior.

Also make it a bit better and reporting errors.

This should actually resolve
https://github.com/vathpela/efibootmgr/issues/12 .

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

Don't error on unset BootOrder when we're trying to add to or rm from it.

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

Produce a slightly more succinct error when we get -EPERM on the disk.

Because one error message is enough.

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

Don't call EPERM "not found" when deleting variables.

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

manpage lists option '--test' which doesn't seem to exist

I'm passing along this bug report from Debian:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=759714

Here's a quick fix:

Add more error messages and better return codes.

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

One more place we're not printing the error code.

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

Fix a segfault in append_extra_args_ascii()

How that worked at all before was beyond me.

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

Make a few more return paths print some modicum of error message.

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