Get rid of the 1024-byte limit on ESP file path.

Not that I think 1024 was something we were going to hit there, but...

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

Handle some other device paths.

Add support for Vendor-Defined Messaging Device Paths as well as
FVFILE() and FVVOL()

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

Get rid of 1024-byte limit for displaying variables as well.

This basically makes all of unparse_path() and its derivatives into
snprintf()-like functions, so we don't have to have static buffers.

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

Add a TODO here so I can keep track of what is yet to be done.

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

Typo fix.

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

Correctly fill out sa_family for MAC address device types.

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

Handle virtio network devices.

Just like virtio scsi devices, look up their bus/device/function by the
virtio device instead of the traditional way.

Signed-off-by: Peter Jones <pjones@redhat.com>
Tested-and-reviewed-by: Laszlo Ersek <lersek@redhat.com>

'-h' is a valid option, recognize it

This completes commit e562a948.

Signed-off-by: Laszlo Ersek <lersek@redhat.com>

Ignore patches.

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

Remove 40-character limit on file paths in /boot.

This is totally unnecessary now; the only real point before was
everything was fixed size to keep it within 1024.

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

Don't allow negative boot numbers.

In the case where you typo the command line and do:

efibootmgr -b -a blah blah blah

instead of

efibootmgr -b 0001 -a blah blah blah

sscanf(..., "%X", &num) happily produces a successful return code and a
value of -10.  This later results in us creating a boot variable named
BootFFFFFFFF6 , which is clearly not okay.  So if we've got a negative
number there, it's probably not what the user wanted.

Bounds check the whole thing.

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

Make make_linux_load_option() slightly more readable.

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

Make add_extra_args_file() work properly.

Two things:
1) make it actually return the data
2) _ascii() and _unicode() will get invoked in some cases, and they need
   to handle the case where they do nothing.

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

Audit strlen() in append_extra_args_ascii() as well.

Similar code, similar bugs.

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

Restore old boot order behavior.

Previously we prepended rather than appending new elements to the boot
order.

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

Do better error checking in read_boot_u16(), and simplify it.

Why am I even doing this stupid bitshifting?

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

Fix byte assignment in read_boot_u16().

It helps if you put the bytes in the right order, and the parenthesis
around the right things to keep them from being all zero.

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

More work on append_extra_args_unicode()

two problems:
1) strlen off-by-one
2) efichar_from_char() awesomely takes a length in bytes, returns in
   bytes, and operates on efi_char16_t sizes.  This can be somewhat
   confusing.

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

You really can't free something you put through realloc().

This should make '-u' work again.

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

Fix typo in arg parsing.

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

Remove some dead code.

If anybody's looking for it, it's in git.

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

Clean up .gitignores some.

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

Make --help also -h, and don't have it exit with an error status.

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

Make tilt_slashes() static.

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

Fix uninitialized variable.

valgrind found a couple of errors, one of which fixes some odd behavior
with BootOrder I was seeing.

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

Fix minor compiler warning.

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

Fix completely wrong compiler warning.

gcc's data flow analysis still kind of blows.

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

Make the error case from make_boot_var() cleaner.

bcl thinks it looks better at the end, and I agree.

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

Don't do unnecessary math.

bcl found this one too - we know len % 5 is 4, because it's tested a few
lines up.

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

Fix error message that says "delete" on the "activate" case.

bcl found this reviewing it.

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

Use libefivar.

Using libefivar divorces us from the kernel api, which is a very good
thing.  It also means we don't need to know about efi_status_t or efi
error types, or carry our own headers for describing variables.

This also makes device paths > 1024 bytes long work (on newer kernels)

Signed-off-by: Peter Jones <pjones@redhat.com>
Conflicts:
src/efibootmgr/efibootmgr.c
src/lib/efi.c

Ignore some files so "git status" is clearer.

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

Silence some minor compiler warnings.

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

Release 0.6.1

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

Modernize "make tarball" for our glorious git future.

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

Prepping for move to GitHub. Updated documentation to refer to new maintainer and new git repository.

From: Colin Watson <cjwatson@ubuntu.com>

Fix -w option to leave an existing unique MBR signature intact. This makes the
implementation match the documentation.

From: Bdale Garbee <bdale@gag.com>

change computeroutput to programlisting to fix man page formatting issue,
update rules to use docbook source for man page, closes Debian bug #680716

Fix unchecked malloc calls.
Resolves: rhbz#715216
From Peter Jones@Redhat

produce virtio-scsi device paths under EDD 3.0
Patch from Laszlo Ersek@RedHat

prevent out-of-bounds access with SCSI_IOCTL_GET_PCI
Patch from Laszlo Ersek@RedHat

disk_get_virt_pci(): generalize to arbitrary major & minor device numbers
Patch from Laszlo Ersek@RedHat

disk_info_from_fd(): group output parameters into struct disk_info
Patch from Laszlo Ersek@RedHat

produce virtio-blk device paths under EDD 3.0
Patch from Laszlo Ersek@RedHat

disk_get_pci(): expose detected interface type
Patch from Laszlo Ersek@RedHat

disk_info_from_fd(): get major/minor from buf.st_rdev
Patch from Laszlo Ersek@RedHat

Add support for UNIX-style path names (forward-slash)
Submitted by Keshav Padram [the.ridikulus.rat@gmail.com]

Update manpage email

Bump efibootmgr version to include latest changes

Add patches from fedora release
* Wed Nov 28 2012 Matthew Garrett <mjg59@srcf.ucam.org> - 0.5.4-14
- efibootmgr-0.5.4-Work-around-broken-Apple-firmware.patch
  Resolves: #873629
- efibootmgr-0.5.4-Remove-device-path-padding-on-non-Itanium.patch - improve
  spec conformance

From: Lane Winner<lane.winner@oracle.com>

make_boot_var does not check for failed status with create_variable.
This can result in a memory leak.
Additionally the user should be notified of the problem.

We encounter this issue on one system after filling up the UEFI boot list
with dummy devices.

The patch fix the problem. It was verified on a Mensa system using RHEL 6.0

Signed-off-by: Yinghai Lu<yinghai@kernel.org>

Handle sector_size != 512.

Disks can have 4kB sectors now, so don't just bail out when that's the
case.

fix disk minor number discovery

Raymund Will noted disk_info_from_fd() incorrectly used logical &&
instead of bitwise & when obtaining the minor number.

Reported in https://bugzilla.novell.com/show_bug.cgi?id=524529#c1

fix minor memory leak

David Binderman noted new_data was being allocated but not freed.  Not
a big deal as the program exits soon thereafter (and is thus freed),
but worth fixing anyhow.

Fixes https://bugzilla.novell.com/show_bug.cgi?id=524529#c1

update ChangeLog

replacing elilo < 3.6-6, not -5

per conversation with clumens.

RPM spec rewrite for Fedora inclusion

efibootmgr is being split out of the elilo SRPM in Fedora, into its
own SRPM.

Makefile cleanups

bump version

take an EXTRA_CFLAGS argument so rpmbuild can give us it's CFLAGS
without overriding ours.

exclude .git and *~ files from the tarball.

src/lib/efi.c: include linux/types.h

patch from Fedora.

make sure GPT_HEADER_SIGNATURE is a 64-bit value

patch from Debian / Ubuntu 0.5.3-3ubuntu1.

avoid more unaligned access warnings

cleanup a few unaligned access warnings

cleanup exit values a little

patch to make efibootmgr have non-zero exit code on errors

We have some automated tools that use efibootmgr which are having a hard
time detecting when efibootmgr fails since it exits with 0 on many
failures.  This patch catches (most) errors and exits with non-zero.  I
also added several error messages for to make it more obvious what was
wrong with the command line arguments.

Signed-off-by: Matt Domsch <Matt_Domsch@dell.com>

add -lz to libs, needed when libpci happens to need it

apply patch from Dave Jiang <djiang@mvista.com> to enable cross-building

v0.5.2.2

document -@ option

make efi_variable_t.DataSize be unsigned long, to match kernel implementation.  This isn't ideal, I'd prefer to be able to have a 32-bit efibootmgr on a 64-bit kernel, but with efi_status_t being a long also, we're already stuck with needing a native copy.

clean up extra_opts_file handling

new option -@ grabs extra variable args from a file

add -g flag, move _FILE_OFFSET_BITS=64 to Makefile

v0.5.2

move _FILE_OFFSET_BITS definition into Makefile

parse PCI domain information in make_net_load_option() on 2.6 kernels

properly free allocated memory (caught by valgrind)

applied patch from Alex Williamson which causes BootXXXX values to be printed in uppercase hex rather than lowercase.  This also now allows the full 64K boot entries, rather than requiring they start with a zero.  Release 0.5.1

put linux.dell.com in spec file

version 0.5.0-test4 released as v0.5.0 with no code changes

Add PCI parent devices to device path for netboot and EDD30 entries

changelog for version 0.5.0-test4

version 0.5.0-test4

Don't let read_boot_order() return EFI_NOT_FOUND when it is about to create it.

Add libpci to libraries linked in

Add Joshua Giles entry

fix reversed logic in create_or_edit_variable(), remove extraneous printks in sysfs_read/write()

Add ability to add/change/delete Timeout, reflect this everywhere.

Remove extra sizeof() in definition of BLKGETSIZE64.  It's
unnecessary, and causes new gcc 3.3.x compilers to complain and fail.

More procfs to sysfs cleanups
Release version 0.5.0-test1

- Seperated access to variables through /proc into efivars_procfs.[ch]
- Added efivars_sysfs.h to access variables through sysfs.
- Moved around some functions, cleaned up some duplication.

* Mon Aug 11 2003 Matt Domsch <Matt_Domsch@dell.com>
- fixed unaligned access errors
- removed extraneous printing of mac addr when creating netboot entries
- sync docbook to README
- whitespace cleanups

Adding filelist-rpm

* Thu Jul 31 2003 Matt Domsch <Matt_Domsch@dell.com>
- Applied patch from Dann Frazier to enable creating netboot entries.
- update AUTHORS with Dann's netboot contribution.
- Until we can get the ACPI HID and UID programatically, make the user
  pass them in when creating a netboot entry.
- Add O_DIRECT support for reading the disk.
- Fix unparse_hardware_path() for the PCI case - the device and function
  values were printed in reverse order.
- Fix the README file to reflect all the options that can be passed, and
  add a new item for netboot entries.
- whitespace cleanups

- Patch from Fabien Lebaillif - Delamare increases the length of the boot
option description from 40 to 64 bytes.

Added efibootmgr.8 man page (generated from efibootmgr.8.docbook
with docbook2man)

* Tue Oct 22 2002 Matt Domsch <Matt_Domsch@dell.com>
- ran docbook2man on the man page Dann Frazier created
- Added man page to spec file
- Added patches from Dann Frazier
  - Removes the requirement to be root to run efibootmgr.  This lets
    mortal users run efibootmgr to check the version, and see the
    --help output. It should also allow efibootmgr to work under
    security systems without the strict root/non-root dichotomy.
    - Checks to see if a boot option was specified when a boot option
    deletion was requested.
- Released version 0.4.1

* Sun Jul 21 2002 Matt Domsch <Matt_Domsch@dell.com>
- Added kernel_has_blkgetsize64() test, now uses BLKGETSIZE64 if on a
  known-good kernel.  This is important when the Large Block Device (64-bit
  block address) patch gets merged into the 2.5 kernel.

moved uname() call into kernel_has_blkgetsize64, renamed that function

added uname test for calling BLKGETSIZE64 ioctl

moved executable to /usr/sbin/

moved location of tarball on domsch.com