Skip to content

Newapi upgrade secboot#154

Merged
mkukri-canonical merged 5 commits into
mainfrom
newapi-upgrade-secboot
Jun 25, 2026
Merged

Newapi upgrade secboot#154
mkukri-canonical merged 5 commits into
mainfrom
newapi-upgrade-secboot

Conversation

@frederic-hoerni

Copy link
Copy Markdown
Contributor

No description provided.

mkukri-canonical and others added 4 commits June 25, 2026 16:09
Packages at the same revision as snapd:
  secboot
  go-tpm2
  go-efilib

snapd reference: 4f657056a2389195c2fa7228a1e0b494dd91c734
The error was:

root@ubuntu-boot-test:~# ./nullbootctl
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x30 pc=0x5dd87f]

goroutine 1 [running]:
github.com/canonical/go-efilib.getVarsBackend({0x0, 0x0})
	.../go/pkg/mod/github.com/canonical/go-efilib@v1.7.1-0.20260310185303-7166aa858b24/vars.go:99 +0x1f
github.com/canonical/go-efilib.ListVariables({0x0, 0x0})
	.../go/pkg/mod/github.com/canonical/go-efilib@v1.7.1-0.20260310185303-7166aa858b24/vars.go:168 +0x25
github.com/canonical/nullboot/efibootmgr.RealEFIVariables.ListVariables(...)
	.../nullboot/efibootmgr/efivars.go:34
github.com/canonical/nullboot/efibootmgr.VariablesSupported(...)
	.../nullboot/efibootmgr/efivars.go:134
github.com/canonical/nullboot/efibootmgr.NewBootManagerForVariables({0x8e1a38?, 0xc000022210?})
	.../nullboot/efibootmgr/bootmgr.go:91 +0xa9
main.main()
	.../nullboot/cmd/nullbootctl/main.go:58 +0x4c5
@mkukri-canonical
mkukri-canonical force-pushed the newapi-upgrade-secboot branch from 2242ac2 to e52bec1 Compare June 25, 2026 15:10
dnephin/pre-commit-golang is archived (last push 2023). Switch to the
maintained golangci/golangci-lint hook, which supersedes the old go-fmt,
go-vet, go-build and go-mod-tidy hooks.
@mkukri-canonical
mkukri-canonical force-pushed the newapi-upgrade-secboot branch from 3f53311 to 4d44874 Compare June 25, 2026 15:21
@frederic-hoerni frederic-hoerni removed their assignment Jun 25, 2026
@mkukri-canonical
mkukri-canonical merged commit e6f5bb1 into main Jun 25, 2026
8 checks passed
@mkukri-canonical
mkukri-canonical deleted the newapi-upgrade-secboot branch June 25, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants