Skip to content

Commit 78ab231

Browse files
committed
fix unchecked errors in cmds/exp/netbootxyz & pkg/boot
replace kexec cmd Signed-off-by: Fabian Wienand <fabian.wienand@9elements.com>
1 parent 9ab6588 commit 78ab231

3 files changed

Lines changed: 15 additions & 4 deletions

File tree

cmds/exp/netbootxyz/netbootxyz.go

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import (
1111
"io"
1212
"net/http"
1313
"os"
14-
"os/exec"
1514
"strings"
1615
"time"
1716

@@ -174,12 +173,15 @@ func (o OSEndpoint) Load() error {
174173
fmt.Printf("With Kernel at %s\n", tmpPath+"vmlinuz")
175174
fmt.Printf("With Initrd at %s\n", tmpPath+"initrd")
176175
fmt.Printf("Commandline: %s\n", o.Commandline)
176+
177177
// Load Kernel and initrd
178178
err = kexec.FileLoad(vmlinuz, initrd, o.Commandline)
179+
if err != nil {
180+
return err
181+
}
182+
179183
// Load KExec kernel and initrd - init cmdline
180-
cmd := exec.Command("kexec", "-e")
181-
err = cmd.Run()
182-
return err
184+
return kexec.Reboot()
183185
}
184186
return err
185187
}

pkg/boot/fit/vfit.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,10 @@ func (s RSASignature) Verify(b []byte, ring openpgp.KeyRing) (*bytes.Reader, err
104104
}
105105

106106
hashed, err := vfile.CalculateHash(bytes.NewReader(b), s.hash.New())
107+
if err != nil {
108+
return r, err
109+
}
110+
107111
for _, key := range keys {
108112
if err = rsa.VerifyPKCS1v15(key, s.hash, hashed, s.value); err == nil {
109113
return r, nil

pkg/boot/uefi/uefi.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@ func checkFVAndGetEntryPoint(name string) (uintptr, error) {
7373
if err != nil {
7474
return 0, err
7575
}
76+
7677
secEntry, err := findSecurityCorePEEntry(dat)
78+
if err != nil {
79+
return 0, err
80+
}
81+
7782
f, err := os.Open(name)
7883
if err != nil {
7984
return 0, err

0 commit comments

Comments
 (0)