Merge pull request #74 from snaka/fix/panic-when-omit-conf-option

fix: `-conf` omitted caused panic by referencing a nil pointer
Songmu · Aug 25, 2023 · b6e383d · b6e383d
2 parents 03cd9a9 + 28abb53
commit b6e383d
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/cmd_apply.go b/cmd_apply.go
@@ -44,6 +44,9 @@ var cmdApply = &runnerImpl{
 				return err
 			}
 		}
+		if c == nil {
+			return errors.New("-conf option required")
+		}
 
 		var ruleNames []string
 		if !*all {

diff --git a/cmd_diff.go b/cmd_diff.go
@@ -44,6 +44,9 @@ var cmdDiff = &runnerImpl{
 				return err
 			}
 		}
+		if c == nil {
+			return errors.New("-conf option required")
+		}
 
 		var ruleNames []string
 		if !*all {

diff --git a/cmd_run.go b/cmd_run.go
@@ -41,6 +41,9 @@ var cmdRun = &runnerImpl{
 				return err
 			}
 		}
+		if c == nil {
+			return errors.New("-conf option required")
+		}
 		ru := c.GetRuleByName(*rule)
 		if ru == nil {
 			return fmt.Errorf("no rules found for %s", *rule)