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

Fix check that mappings can only have storage location

Browse Source
This commit is contained in:
Mathias Baumann 2019-02-05 15:51:19 +01:00
parent d7e2838702
commit 77f407d450
6 changed files with 19 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Changelog.md
View File

@ -3,6 +3,7 @@
Bugfixes: Bugfixes:
* Code generator: Defensively pad allocation of creationCode and runtimeCode to multiples of 32 bytes. * Code generator: Defensively pad allocation of creationCode and runtimeCode to multiples of 32 bytes.
* Parser: Disallow empty import statements. * Parser: Disallow empty import statements.
* Type Checker: Dissallow mappings with data locations other than 'storage'
* Type system: Properly report packed encoded size for arrays and structs (mostly unused until now). * Type system: Properly report packed encoded size for arrays and structs (mostly unused until now).

15
libsolidity/analysis/TypeChecker.cpp
View File

@ -334,10 +334,17 @@ bool TypeChecker::visit(FunctionDefinition const& _function)
auto checkArgumentAndReturnParameter = [&](VariableDeclaration const& var) { auto checkArgumentAndReturnParameter = [&](VariableDeclaration const& var) {
if (type(var)->category() == Type::Category::Mapping) if (type(var)->category() == Type::Category::Mapping)
{ {
if (!type(var)->dataStoredIn(DataLocation::Storage)) if (var.referenceLocation() != VariableDeclaration::Location::Storage)
m_errorReporter.typeError(var.location(), "Mapping types can only have a data location of \"storage\"." ); {
else if (!isLibraryFunction && _function.isPublic()) if (!isLibraryFunction && _function.isPublic())
m_errorReporter.typeError(var.location(), "Mapping types for parameters or return variables can only be used in internal or library functions."); m_errorReporter.typeError(var.location(), "Mapping types can only have a data location of \"storage\" and thus only be parameters or return variables for internal or library functions.");
else
m_errorReporter.typeError(var.location(), "Mapping types can only have a data location of \"storage\"." );
}
else
{
solAssert(isLibraryFunction || !_function.isPublic(), "Mapping types for parameters or return variables can only be used in internal or library functions.");
}
} }
else else
{ {

4
test/libsolidity/syntaxTests/types/mapping/mapping_data_location_function_param_external.sol
View File

@ -2,5 +2,5 @@ contract c {
function f1(mapping(uint => uint) calldata) pure external returns (mapping(uint => uint) memory) {} function f1(mapping(uint => uint) calldata) pure external returns (mapping(uint => uint) memory) {}
} }
// ---- // ----
// TypeError: (29-59): Mapping types for parameters or return variables can only be used in internal or library functions. // TypeError: (29-59): Mapping types can only have a data location of "storage" and thus only be parameters or return variables for internal or library functions.
// TypeError: (84-112): Mapping types for parameters or return variables can only be used in internal or library functions. // TypeError: (84-112): Mapping types can only have a data location of "storage" and thus only be parameters or return variables for internal or library functions.

4
test/libsolidity/syntaxTests/types/mapping/mapping_data_location_function_param_internal.sol
View File

@ -1,4 +1,6 @@
contract c { contract c {
function f4(mapping(uint => uint) memory) pure internal {} function f4(mapping(uint => uint) storage) pure internal {}
function f5(mapping(uint => uint) memory) pure internal {}
} }
// ---- // ----
// TypeError: (93-121): Mapping types can only have a data location of "storage".

2
test/libsolidity/syntaxTests/types/mapping/mapping_data_location_function_param_public.sol
View File

@ -2,4 +2,4 @@ contract c {
function f3(mapping(uint => uint) memory) view public {} function f3(mapping(uint => uint) memory) view public {}
} }
// ---- // ----
// TypeError: (29-57): Mapping types for parameters or return variables can only be used in internal or library functions. // TypeError: (29-57): Mapping types can only have a data location of "storage" and thus only be parameters or return variables for internal or library functions.

2
test/libsolidity/syntaxTests/types/mapping/mapping_return_public_memory.sol
View File

@ -3,4 +3,4 @@ contract C {
} }
} }
// ---- // ----
// TypeError: (51-79): Mapping types for parameters or return variables can only be used in internal or library functions. // TypeError: (51-79): Mapping types can only have a data location of "storage" and thus only be parameters or return variables for internal or library functions.