Merge pull request #238 from cloudbase/fix-nil-pointer-dereference

Check for nil pointer before dereferencing
cloudbase · Mar 26, 2024 · 18bb126 · 18bb126
2 parents bb67ae5 + e2eb8f7
commit 18bb126
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/cmd/garm-cli/cmd/log.go b/cmd/garm-cli/cmd/log.go
@@ -46,10 +46,16 @@ var logCmd = &cobra.Command{
 		if err != nil {
 			var resp apiParams.APIErrorResponse
 			var msg string
-			if err := json.NewDecoder(response.Body).Decode(&resp); err == nil {
-				msg = resp.Details
+			var status string
+			if response != nil {
+				if response.Body != nil {
+					if err := json.NewDecoder(response.Body).Decode(&resp); err == nil {
+						msg = resp.Details
+					}
+				}
+				status = response.Status
 			}
-			log.Fatalf("failed to stream logs: %s (%s)", msg, response.Status)
+			log.Fatalf("failed to stream logs: %q %s (%s)", err, msg, status)
 		}
 		defer c.Close()