Skip to content

Feature/subcommand key recovery#155

Open
frederic-hoerni wants to merge 5 commits into
mainfrom
feature/subcommand-key-recovery
Open

Feature/subcommand key recovery#155
frederic-hoerni wants to merge 5 commits into
mainfrom
feature/subcommand-key-recovery

Conversation

@frederic-hoerni
Copy link
Copy Markdown

No description provided.

kukrimate and others added 5 commits April 3, 2026 16:47
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
Things missing so far:
- unit testing of new functions
- complete functional testing of nullboot (non regression on other
  functions)
@frederic-hoerni frederic-hoerni self-assigned this May 19, 2026
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 26085281643

Coverage decreased (-7.0%) to 63.776%

Details

  • Coverage decreased (-7.0%) from the base build.
  • Patch coverage: 155 uncovered changes across 4 files (41 of 196 lines covered, 20.92%).
  • 4 coverage regressions across 1 file.

Uncovered Changes

File Changed Covered %
cmd/nullbootctl/main.go 60 0 0.0%
luks2/luks2.go 46 0 0.0%
efibootmgr/reseal.go 81 40 49.38%
efibootmgr/efivars.go 9 1 11.11%

Coverage Regressions

4 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
efibootmgr/efivars.go 4 80.0%

Coverage Stats

Coverage Status
Relevant Lines: 1430
Covered Lines: 912
Line Coverage: 63.78%
Coverage Strength: 1.46 hits per line

💛 - Coveralls

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.

3 participants