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 ICE caused by an array of mappings

Browse Source
This commit is contained in:
a3d4 2020-09-27 21:35:41 +02:00
parent 343c13f94c
commit 3c876fcb60
4 changed files with 32 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Changelog.md
View File

@ -4,6 +4,10 @@ Compiler Features:
* SMTChecker: Support ``addmod`` and ``mulmod``.
Bugfixes:
* Type Checker: Fix internal compiler error when calling `.push(<arg>)` for a storage array with a nested mapping.
### 0.7.2 (2020-09-28)
Important Bugfixes:

12
libsolidity/analysis/TypeChecker.cpp
View File

@ -2684,10 +2684,20 @@ bool TypeChecker::visit(MemberAccess const& _memberAccess)
"Using \"." + memberName + "(...)\" is deprecated. Use \"{" + memberName + ": ...}\" instead."
);
if (
funType->kind() == FunctionType::Kind::ArrayPush &&
arguments.value().numArguments() != 0 &&
exprType->containsNestedMapping()
)
m_errorReporter.typeError(
8871_error,
_memberAccess.location(),
"Storage arrays with nested mappings do not support .push(<arg>)."
);
if (!funType->bound())
if (auto contractType = dynamic_cast<ContractType const*>(exprType))
requiredLookup = contractType->isSuper() ? VirtualLookup::Super : VirtualLookup::Virtual;
}
annotation.requiredLookup = requiredLookup;

7
test/libsolidity/syntaxTests/array/pop/storage_with_mapping_pop.sol Normal file
View File

@ -0,0 +1,7 @@
contract C {
mapping(uint=>uint)[] array;
mapping(uint=>uint) map;
function f() public {
array.pop();
}
}

10
test/libsolidity/syntaxTests/array/push/storage_with_mapping_push.sol Normal file
View File

@ -0,0 +1,10 @@
contract C {
mapping(uint=>uint)[] array;
mapping(uint=>uint) map;
function f() public {
array.push();
array.push(map);
}
}
// ----
// TypeError 8871: (131-141): Storage arrays with nested mappings do not support .push(<arg>).