From 4f3f09fec500071cd8dadecf5319073b9c1f5d60 Mon Sep 17 00:00:00 2001 From: "Brian C. Lane" Date: Fri, 15 May 2020 14:54:42 -0700 Subject: [PATCH] worker: Check OSBuildOutput for nil before using it It is possible for it to end up as nil so it needs to be checked everywhere it is used. --- internal/worker/server.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/worker/server.go b/internal/worker/server.go index 0024d73d2..38c2294db 100644 --- a/internal/worker/server.go +++ b/internal/worker/server.go @@ -103,7 +103,7 @@ func (s *Server) JobStatus(id uuid.UUID) (*JobStatus, error) { if canceled { state = common.CFailed } else if !finished.IsZero() { - if result.OSBuildOutput.Success { + if result.OSBuildOutput != nil && result.OSBuildOutput.Success { state = common.CFinished } else { state = common.CFailed