job parsing: fix panic when variable validation is missing condition

hashicorp · Feb 1, 2023 · ddbd9c3 · ddbd9c3
1 parent e4e4dc1
commit ddbd9c3
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/jobspec2/types.variables.go b/jobspec2/types.variables.go
@@ -95,6 +95,17 @@ func (v *Variable) validateValue(val VariableAssignment) (diags hcl.Diagnostics)
 	for _, validation := range v.Validations {
 		const errInvalidCondition = "Invalid variable validation result"
 
+		if validation.Condition == nil {
+			diags = append(diags, &hcl.Diagnostic{
+				Severity:    hcl.DiagError,
+				Summary:     "Invalid variable validation specification",
+				Detail:      "validation requires a condition.",
+				Subject:     validation.DeclRange.Ptr(),
+				EvalContext: hclCtx,
+			})
+			continue
+		}
+
 		result, moreDiags := validation.Condition.Value(hclCtx)
 		diags = append(diags, moreDiags...)
 		if !result.IsKnown() {