[Bug 1937115] Re: Unable to boot/install Impish daily in UEFI boot mode

[Brian Murray]

Hello Leó, or anyone else affected,

Accepted debian-installer into bionic-proposed. The package will build
now and be available at https://launchpad.net/ubuntu/+source/debian-
installer/20101020ubuntu543.19 in a few hours, and then in the -proposed
repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how
to enable and use -proposed.  Your feedback will aid us getting this
update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested, what testing has been
performed on the package and change the tag from verification-needed-
bionic to verification-done-bionic. If it does not fix the bug for you,
please add a comment stating that, and change the tag to verification-
failed-bionic. In either case, without details of your testing we will
not be able to proceed.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance for helping!

N.B. The updated package will be released to -updates after the bug(s)
fixed by this package have been verified and the package has been in
-proposed for a minimum of 7 days.

** Changed in: debian-installer (Ubuntu Bionic)
       Status: In Progress => Fix Committed

** Tags removed: verification-done verification-done-bionic
** Tags added: verification-needed verification-needed-bionic

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to shim in Ubuntu.
https://bugs.launchpad.net/bugs/1937115

Title:
  Unable to boot/install Impish daily in UEFI boot mode

Status in OEM Priority Project:
  Fix Released
Status in cd-boot-images-amd64 package in Ubuntu:
  Fix Released
Status in debian-installer package in Ubuntu:
  Invalid
Status in shim package in Ubuntu:
  Fix Released
Status in shim-signed package in Ubuntu:
  Fix Released
Status in cd-boot-images-amd64 source package in Xenial:
  Invalid
Status in debian-installer source package in Xenial:
  Invalid
Status in shim source package in Xenial:
  New
Status in shim-signed source package in Xenial:
  New
Status in cd-boot-images-amd64 source package in Bionic:
  Invalid
Status in debian-installer source package in Bionic:
  Fix Committed
Status in shim source package in Bionic:
  Fix Released
Status in shim-signed source package in Bionic:
  Fix Released
Status in shim source package in Focal:
  Fix Released
Status in shim-signed source package in Focal:
  Fix Released
Status in shim source package in Hirsute:
  Fix Released
Status in shim-signed source package in Hirsute:
  Fix Released
Status in cd-boot-images-amd64 source package in Impish:
  Fix Released
Status in debian-installer source package in Impish:
  Invalid
Status in shim source package in Impish:
  Fix Released
Status in shim-signed source package in Impish:
  Fix Released

Bug description:
  [Impact]
  Removable media fail to boot on some machines due to garbage in firmware generated boot entries. This is a regression from LP: #1929471 introduced in shim 15.4-0ubuntu7.

  In addition we also cherry-pick
  https://github.com/rhboot/shim/pull/365 to fix a potential buffer
  overflow.

  In addition we also cherry-pick
  https://github.com/rhboot/shim/pull/396 to fix possible firmware
  corruption in the fallback loader due to out-of-bounds access to the
  boot order array.

  [Test plan]
  I'm sure someone can test a new daily impish ISO on an affected machine. Verification is not necessary for individual SRU releases.

  We will also do boot of an installed system, and perform the usual
  chainloaded netbooting test, prior to uploading.

  [Regression potential]

  We'll apply four patches for the main issue:

  Patches 1/2 (https://github.com/rhboot/shim/pull/393) add a fallback
  to the default loader (with 2s timeout) if we failed to load any
  specified loader. This ensures we can always boot our default loader,
  and hence installed systems, even if there is garbage around. It also
  adds debugging output for the load option that is being parsed, such
  that people can debug why it failed.

  Patches 3 and 4 (https://github.com/rhboot/shim/pull/399) disables
  parsing load options entirely when booting via the removable media
  path (boot*.efi). This means that any system where admins generate
  boot entries like bootx64.efi fwupdx64.efi to load a specific second
  stage will fail to load that second stage and load grubx64.efi
  instead.

  is this a problem in practice?

  On installed systems, we install \EFI\BOOT\bootx64.efi in addition to
  \EFI\ubuntu\shimx64.efi, and generate boot loader entries for the
  latter. The former will always use the fbx64.efi fallback loader to
  add missing boot loader entries and load grub, hence it doesn't seem
  to support custom options anyway (you'd have to delete fbx64.efi; and
  even then - you still have the shimx64.efi binary).

  On installer media, we do not expect you to specify another second
  stage loader anyway. It's arguably only problematic there, as those
  could be read-only and hence you can't rename the binary to
  shimx64.efi.

  For the overflow, the fix is trivially correct.

  For the fallback loader, the fix is reasonably trivial; regression
  potential there could be failure to add a boot entry which is not
  super critical.

  [Original bug report]
  Testing Ubuntu Impish daily ISO= http://cdimage.ubuntu.com/daily-live/20210721/impish-desktop-amd64.iso

  The test machines are:

  1. Dell [ Optiplex] MT 7040 i7-6700 booting in UEFI mode
  2. Dell [ Optiplex] MT 7060, i7-8700 booting in UEFI+secure boot mode

  Boot from USB media fails with the message:

  Failed to open /EFI/boot/? invalid parameter

  No further info available

To manage notifications about this bug go to:
https://bugs.launchpad.net/oem-priority/+bug/1937115/+subscriptions