Skip to content

Commit 74dd874

Browse files
author
Chris Gilmer
committed
Check errors on closing or removing files
1 parent e1f78df commit 74dd874

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

cmd/main.go

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -450,7 +450,7 @@ func generateQrCode(payload string, tempFile *os.File) error {
450450

451451
// Write the QR PNG to the Temp File
452452
if _, err := tempFile.Write(qr); err != nil {
453-
tempFile.Close()
453+
_ = tempFile.Close()
454454
return err
455455
}
456456
return nil
@@ -542,7 +542,12 @@ func main() {
542542
log.Fatal(err)
543543
}
544544
// Cleanup after ourselves
545-
defer os.Remove(tempfile.Name())
545+
defer func() {
546+
errRemove := os.Remove(tempfile.Name())
547+
if errRemove != nil {
548+
log.Fatal(errRemove)
549+
}
550+
}()
546551

547552
config, err := vault.LoadConfigFromEnv()
548553
if err != nil {

0 commit comments

Comments
 (0)