Protect against possible panic in onBuildFileError

VanDucTan · Apr 5, 2024 · 091db00 · 091db00
1 parent caff7c5
commit 091db00
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/app/server/model/plan/build_finish.go b/app/server/model/plan/build_finish.go
@@ -303,10 +303,12 @@ func (fileState *activeBuildStreamFileState) onBuildFileError(err error) {
 	activeBuild.Success = false
 	activeBuild.Error = err
 
-	activePlan.StreamDoneCh <- &shared.ApiError{
-		Type:   shared.ApiErrorTypeOther,
-		Status: http.StatusInternalServerError,
-		Msg:    err.Error(),
+	if activePlan != nil {
+		activePlan.StreamDoneCh <- &shared.ApiError{
+			Type:   shared.ApiErrorTypeOther,
+			Status: http.StatusInternalServerError,
+			Msg:    err.Error(),
+		}
 	}
 
 	if err != nil {