diff options
| author | Leonardo Alt <leo@ethereum.org> | 2018-07-31 17:44:39 +0800 |
|---|---|---|
| committer | Leonardo Alt <leo@ethereum.org> | 2018-08-16 17:59:41 +0800 |
| commit | 583e7156ba214c98096217d18f6270e4a46491d9 (patch) | |
| tree | af71c85a7044eecb09bcdb329be5923b4bc9f105 /libsolidity/analysis | |
| parent | cc6fa6d61fb934617d088bb04766ef0dea614b4f (diff) | |
| download | dexon-solidity-583e7156ba214c98096217d18f6270e4a46491d9.tar.gz dexon-solidity-583e7156ba214c98096217d18f6270e4a46491d9.tar.zst dexon-solidity-583e7156ba214c98096217d18f6270e4a46491d9.zip | |
Dynamic type as mapping key returns error instead of assertion fail
Diffstat (limited to 'libsolidity/analysis')
| -rw-r--r-- | libsolidity/analysis/TypeChecker.cpp | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/libsolidity/analysis/TypeChecker.cpp b/libsolidity/analysis/TypeChecker.cpp index 5033fd63..fca64f6a 100644 --- a/libsolidity/analysis/TypeChecker.cpp +++ b/libsolidity/analysis/TypeChecker.cpp @@ -819,7 +819,9 @@ bool TypeChecker::visit(VariableDeclaration const& _variable) ) m_errorReporter.typeError(_variable.location(), "Internal or recursive type is not allowed for public state variables."); - if (varType->category() == Type::Category::Array) + switch (varType->category()) + { + case Type::Category::Array: if (auto arrayType = dynamic_cast<ArrayType const*>(varType.get())) if ( ((arrayType->location() == DataLocation::Memory) || @@ -827,6 +829,18 @@ bool TypeChecker::visit(VariableDeclaration const& _variable) !arrayType->validForCalldata() ) m_errorReporter.typeError(_variable.location(), "Array is too large to be encoded."); + break; + case Type::Category::Mapping: + if (auto mappingType = dynamic_cast<MappingType const*>(varType.get())) + if ( + mappingType->keyType()->isDynamicallySized() && + _variable.visibility() == Declaration::Visibility::Public + ) + m_errorReporter.typeError(_variable.location(), "Dynamically-sized keys for public mappings are not supported."); + break; + default: + break; + } return false; } |