cerc-io/solidity
16
0
Fork 0
mirror of https://github.com/ethereum/solidity synced 2023-10-03 13:03:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

AsmParser: disallow trailing commas in function call arguments.

Browse Source
This commit is contained in:
Daniel Kirchner 2019-06-06 13:16:27 +02:00
parent e7e700be38
commit 6368cd4c82
5 changed files with 29 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Changelog.md
View File

@ -9,6 +9,7 @@ Compiler Features:
Bugfixes:
* Yul / Inline Assembly Parser: Disallow trailing commas in function call arguments.
Build System:

10
libyul/AsmParser.cpp
View File

@ -609,12 +609,14 @@ Expression Parser::parseCall(Parser::ElementaryOperation&& _initialOp)
else
ret = std::move(boost::get<FunctionCall>(_initialOp));
expectToken(Token::LParen);
while (currentToken() != Token::RParen)
if (currentToken() != Token::RParen)
{
ret.arguments.emplace_back(parseExpression());
if (currentToken() == Token::RParen)
break;
expectToken(Token::Comma);
while (currentToken() != Token::RParen)
{
expectToken(Token::Comma);
ret.arguments.emplace_back(parseExpression());
}
}
ret.location.end = endPosition();
expectToken(Token::RParen);

3
test/libsolidity/syntaxTests/inlineAssembly/invalid/empty_fun_arg.sol → test/libsolidity/syntaxTests/inlineAssembly/invalid/empty_fun_arg_beginning.sol
View File

@ -3,10 +3,9 @@ contract C {
assembly {
function f(a, b) {}
f()
f(1,)
f(,1)
}
}
}
// ----
// ParserError: (113-114): Literal, identifier or instruction expected.
// ParserError: (101-102): Literal, identifier or instruction expected.

11
test/libsolidity/syntaxTests/inlineAssembly/invalid/empty_fun_arg_end.sol Normal file
View File

@ -0,0 +1,11 @@
contract C {
function f() public pure {
assembly {
function f(a, b) {}
f()
f(1,)
}
}
}
// ----
// ParserError: (103-104): Literal, identifier or instruction expected.

10
test/libsolidity/syntaxTests/inlineAssembly/invalid/empty_fun_arg_middle.sol Normal file
View File

@ -0,0 +1,10 @@
contract C {
function f() public pure {
assembly {
function f(a, b, c) {}
f(1,,1)
}
}
}
// ----
// ParserError: (96-97): Literal, identifier or instruction expected.