Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Variadic math functions do not validate number of args #10827

Closed
jmooring opened this issue Mar 12, 2023 · 2 comments · Fixed by #10828
Closed

Variadic math functions do not validate number of args #10827

jmooring opened this issue Mar 12, 2023 · 2 comments · Fixed by #10828

Comments

@jmooring
Copy link
Member

  v0.111.2 v0.111.3
{{ add 1 }} wrong number of args for add: want 2 got 1 no error
{{ sub 1 }} wrong number of args for sub: want 2 got 1 no error
{{ mul 1 }} wrong number of args for mul: want 2 got 1 no error
{{ div 1 }} wrong number of args for div: want 2 got 1 no error
{{ math.Max 1 }} wrong number of args for Max: want 2 got 1 no error
{{ math.Min 1 }} wrong number of args for Min: want 2 got 1 no error

cc @septs

@septs
Copy link
Contributor

septs commented Mar 13, 2023

@jmooring

@bep bep removed the NeedsTriage label Mar 13, 2023
@bep bep added this to the v0.112.0 milestone Mar 13, 2023
bep pushed a commit that referenced this issue Mar 14, 2023
@github-actions
Copy link

github-actions bot commented Apr 5, 2023

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants