-
Notifications
You must be signed in to change notification settings - Fork 17
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
efi: Make use of the grub's prefix for detection
Avoid relying on the signature in order to detect an Ubuntu production grub binary for binaries without a SBAT section, as this breaks test cases in snapd where grub is re-signed. Instead, obtain the prefix from the "mods" section (which is set to "/EFI/ubuntu" in Ubuntu).
- Loading branch information
1 parent
06ca8cc
commit 313b997
Showing
9 changed files
with
285 additions
and
9 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,158 @@ | ||
| // -*- Mode: Go; indent-tabs-mode: t -*- | ||
|
|
||
| /* | ||
| * Copyright (C) 2023 Canonical Ltd | ||
| * | ||
| * This program is free software: you can redistribute it and/or modify | ||
| * it under the terms of the GNU General Public License version 3 as | ||
| * published by the Free Software Foundation. | ||
| * | ||
| * This program is distributed in the hope that it will be useful, | ||
| * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
| * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
| * GNU General Public License for more details. | ||
| * | ||
| * You should have received a copy of the GNU General Public License | ||
| * along with this program. If not, see <http://www.gnu.org/licenses/>. | ||
| * | ||
| */ | ||
|
|
||
| package efi | ||
|
|
||
| import ( | ||
| "encoding/binary" | ||
| "errors" | ||
| "fmt" | ||
| "io" | ||
| "math" | ||
|
|
||
| pe "github.com/snapcore/secboot/internal/pe1.14" | ||
| ) | ||
|
|
||
| const grubObjTypePrefix uint32 = 3 | ||
|
|
||
| type grubModuleHeader struct { | ||
| Type uint32 | ||
| Size uint32 | ||
| } | ||
|
|
||
| const grubModuleMagic uint32 = 0x676d696d | ||
|
|
||
| type grubModuleInfo32 struct { | ||
| Magic uint32 | ||
| Offset uint32 | ||
| Size uint32 | ||
| } | ||
|
|
||
| type grubModuleInfo64 struct { | ||
| Magic uint32 | ||
| Padding uint32 | ||
| Offset uint64 | ||
| Size uint64 | ||
| } | ||
|
|
||
| type grubModule struct { | ||
| Type uint32 | ||
| *io.SectionReader | ||
| } | ||
|
|
||
| type grubImageHandle interface { | ||
| peImageHandle | ||
|
|
||
| Prefix() (string, error) | ||
| } | ||
|
|
||
| type grubImageHandleImpl struct { | ||
| peImageHandle | ||
| } | ||
|
|
||
| // newGrubImageHandle returns a new grubImageHandle for the supplied peImageHandle. | ||
| var newGrubImageHandle = func(image peImageHandle) grubImageHandle { | ||
| return &grubImageHandleImpl{peImageHandle: image} | ||
| } | ||
|
|
||
| func (h *grubImageHandleImpl) mods() ([]grubModule, error) { | ||
| section := h.OpenSection("mods") | ||
| if section == nil { | ||
| return nil, errors.New("no mods section") | ||
| } | ||
|
|
||
| var r *io.SectionReader | ||
| switch h.Machine() { | ||
| case pe.IMAGE_FILE_MACHINE_AMD64, pe.IMAGE_FILE_MACHINE_ARM64, pe.IMAGE_FILE_MACHINE_RISCV64: | ||
| var info grubModuleInfo64 | ||
| if err := binary.Read(section, binary.LittleEndian, &info); err != nil { | ||
| return nil, fmt.Errorf("cannot obtain modules info: %w", err) | ||
| } | ||
| if info.Magic != grubModuleMagic { | ||
| return nil, errors.New("invalid modules magic") | ||
| } | ||
| if info.Size > math.MaxInt64 { | ||
| return nil, errors.New("invalid modules offset") | ||
| } | ||
| if info.Size > math.MaxInt64 || info.Size < uint64(binary.Size(info)) { | ||
| return nil, errors.New("invalid modules size") | ||
| } | ||
| r = io.NewSectionReader(section, int64(info.Offset), int64(info.Size)-int64(binary.Size(info))) | ||
| case pe.IMAGE_FILE_MACHINE_ARM, pe.IMAGE_FILE_MACHINE_I386, pe.IMAGE_FILE_MACHINE_RISCV32: | ||
| var info grubModuleInfo32 | ||
| if err := binary.Read(section, binary.LittleEndian, &info); err != nil { | ||
| return nil, fmt.Errorf("cannot obtain modules info: %w", err) | ||
| } | ||
| if info.Magic != grubModuleMagic { | ||
| return nil, errors.New("invalid module magic") | ||
| } | ||
| if info.Size < uint32(binary.Size(info.Size)) { | ||
| return nil, errors.New("invalid modules size") | ||
| } | ||
| r = io.NewSectionReader(section, int64(info.Offset), int64(info.Size)-int64(binary.Size(info))) | ||
| default: | ||
| return nil, fmt.Errorf("unrecognized machine: %d", h.Machine()) | ||
| } | ||
|
|
||
| var mods []grubModule | ||
|
|
||
| for { | ||
| var hdr grubModuleHeader | ||
| if err := binary.Read(r, binary.LittleEndian, &hdr); err != nil { | ||
| if err == io.EOF { | ||
| break | ||
| } | ||
| return nil, fmt.Errorf("cannot obtain module header: %w", err) | ||
| } | ||
|
|
||
| offset, _ := r.Seek(0, io.SeekCurrent) | ||
| size := int64(hdr.Size) - int64(binary.Size(hdr)) | ||
| mods = append(mods, grubModule{ | ||
| Type: hdr.Type, | ||
| SectionReader: io.NewSectionReader(r, offset, size), | ||
| }) | ||
|
|
||
| if _, err := r.Seek(size, io.SeekCurrent); err != nil { | ||
| return nil, fmt.Errorf("cannot seek to next module: %w", err) | ||
| } | ||
| } | ||
|
|
||
| return mods, nil | ||
| } | ||
|
|
||
| func (h *grubImageHandleImpl) Prefix() (string, error) { | ||
| mods, err := h.mods() | ||
| if err != nil { | ||
| return "", err | ||
| } | ||
|
|
||
| for _, mod := range mods { | ||
| if mod.Type != grubObjTypePrefix { | ||
| continue | ||
| } | ||
|
|
||
| prefix, err := io.ReadAll(newCstringReader(mod)) | ||
| if err != nil { | ||
| return "", fmt.Errorf("cannot obtain prefix: %w", err) | ||
| } | ||
| return string(prefix), nil | ||
| } | ||
|
|
||
| return "", nil | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.