fix panic when arity is unlimited and there is no argument

chaisql · Nov 21, 2023 · 31c0128 · 31c0128
1 parent 8422929
commit 31c0128
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/internal/expr/functions/definition.go b/internal/expr/functions/definition.go
@@ -60,6 +60,9 @@ func (fd *definition) Name() string {
 }
 
 func (fd *definition) Function(args ...expr.Expr) (expr.Function, error) {
+	if fd.arity == UNLIMITED && len(args) == 0 {
+		return nil, fmt.Errorf("%s() requires at least one argument", fd.name)
+	}
 	if fd.arity != UNLIMITED && (len(args) != fd.arity) {
 		return nil, fmt.Errorf("%s() takes %d argument(s), not %d", fd.name, fd.arity, len(args))
 	}