Skip to content

Commit 18bb126

Browse files
Merge pull request #238 from cloudbase/fix-nil-pointer-dereference
Check for nil pointer before dereferencing
2 parents bb67ae5 + e2eb8f7 commit 18bb126

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

cmd/garm-cli/cmd/log.go

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,16 @@ var logCmd = &cobra.Command{
4646
if err != nil {
4747
var resp apiParams.APIErrorResponse
4848
var msg string
49-
if err := json.NewDecoder(response.Body).Decode(&resp); err == nil {
50-
msg = resp.Details
49+
var status string
50+
if response != nil {
51+
if response.Body != nil {
52+
if err := json.NewDecoder(response.Body).Decode(&resp); err == nil {
53+
msg = resp.Details
54+
}
55+
}
56+
status = response.Status
5157
}
52-
log.Fatalf("failed to stream logs: %s (%s)", msg, response.Status)
58+
log.Fatalf("failed to stream logs: %q %s (%s)", err, msg, status)
5359
}
5460
defer c.Close()
5561

0 commit comments

Comments
 (0)