From b6264311b06753e93f04e1d00c1b0b2bc2472760 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20Henrique=20Guard=C3=A3o=20Gandarez?=
 <gandarez@gmail.com>
Date: Tue, 15 Aug 2023 16:09:26 -0300
Subject: [PATCH] Fix nil pointer dereference when catching panic from bbolt

---
 pkg/offline/offline.go | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/pkg/offline/offline.go b/pkg/offline/offline.go
index 18612f09..c5d6c119 100644
--- a/pkg/offline/offline.go
+++ b/pkg/offline/offline.go
@@ -7,7 +7,6 @@ import (
 	"math"
 	"net/http"
 	"path/filepath"
-	"runtime/debug"
 	"time"
 
 	"github.com/wakatime/wakatime-cli/pkg/api"
@@ -398,18 +397,15 @@ func ReadHeartbeats(filepath string, limit int) ([]heartbeat.Heartbeat, error) {
 
 // openDB opens a connection to the offline db.
 // It returns the pointer to bolt.DB, a function to close the connection and an error.
-func openDB(filepath string) (*bolt.DB, func(), error) {
-	var err error
-
+// Although named parameters should be avoided, this func uses them to access inside the deferred function and set an error.
+func openDB(filepath string) (db *bolt.DB, _ func(), err error) {
 	defer func() {
 		if r := recover(); r != nil {
 			err = ErrOpenDB{Err: fmt.Errorf("panicked: %v", r)}
-
-			log.Errorf("%w. Stack: %s", err, string(debug.Stack()))
 		}
 	}()
 
-	db, err := bolt.Open(filepath, 0600, &bolt.Options{Timeout: 30 * time.Second})
+	db, err = bolt.Open(filepath, 0600, &bolt.Options{Timeout: 30 * time.Second})
 	if err != nil {
 		return nil, nil, fmt.Errorf("failed to open db file: %s", err)
 	}