diff options
| author | Alex Beregszaszi <alex@rtfs.hu> | 2016-08-27 03:01:47 +0800 |
|---|---|---|
| committer | Alex Beregszaszi <alex@rtfs.hu> | 2016-09-06 18:40:00 +0800 |
| commit | 6ec40b3cdeca667cea2222755c6e145c72279b1d (patch) | |
| tree | 090d1e61bf3d1f8807c0261ac05a441b8710587e | |
| parent | f687635e4743a1e12d01fb082a4f8a76a759ab48 (diff) | |
| download | dexon-solidity-6ec40b3cdeca667cea2222755c6e145c72279b1d.tar.gz dexon-solidity-6ec40b3cdeca667cea2222755c6e145c72279b1d.tar.zst dexon-solidity-6ec40b3cdeca667cea2222755c6e145c72279b1d.zip | |
Reject constant modifier on the fallback function
| -rw-r--r-- | libsolidity/analysis/TypeChecker.cpp | 2 | ||||
| -rw-r--r-- | libsolidity/interface/InterfaceHandler.cpp | 1 | ||||
| -rw-r--r-- | test/libsolidity/SolidityABIJSON.cpp | 2 | ||||
| -rw-r--r-- | test/libsolidity/SolidityNameAndTypeResolution.cpp | 11 |
4 files changed, 13 insertions, 3 deletions
diff --git a/libsolidity/analysis/TypeChecker.cpp b/libsolidity/analysis/TypeChecker.cpp index 06e26a76..fc1b1f59 100644 --- a/libsolidity/analysis/TypeChecker.cpp +++ b/libsolidity/analysis/TypeChecker.cpp @@ -94,6 +94,8 @@ bool TypeChecker::visit(ContractDefinition const& _contract) fallbackFunction = function; if (_contract.isLibrary()) typeError(fallbackFunction->location(), "Libraries cannot have fallback functions."); + if (fallbackFunction->isDeclaredConst()) + typeError(fallbackFunction->location(), "Fallback function cannot be declared constant."); if (!fallbackFunction->parameters().empty()) typeError(fallbackFunction->parameterList().location(), "Fallback function cannot take parameters."); if (!fallbackFunction->returnParameters().empty()) diff --git a/libsolidity/interface/InterfaceHandler.cpp b/libsolidity/interface/InterfaceHandler.cpp index 6e3ae78f..de16a372 100644 --- a/libsolidity/interface/InterfaceHandler.cpp +++ b/libsolidity/interface/InterfaceHandler.cpp @@ -81,7 +81,6 @@ string InterfaceHandler::abiInterface(ContractDefinition const& _contractDef) solAssert(!!externalFunctionType, ""); Json::Value method; method["type"] = "fallback"; - method["constant"] = externalFunctionType->isConstant(); method["payable"] = externalFunctionType->isPayable(); abi.append(method); } diff --git a/test/libsolidity/SolidityABIJSON.cpp b/test/libsolidity/SolidityABIJSON.cpp index 185ba3bf..073d7d97 100644 --- a/test/libsolidity/SolidityABIJSON.cpp +++ b/test/libsolidity/SolidityABIJSON.cpp @@ -644,7 +644,6 @@ BOOST_AUTO_TEST_CASE(include_fallback_function) char const* interface = R"( [ { - "constant" : false, "payable": false, "type" : "fallback" } @@ -696,7 +695,6 @@ BOOST_AUTO_TEST_CASE(payable_fallback_unction) char const* interface = R"( [ { - "constant" : false, "payable": true, "type" : "fallback" } diff --git a/test/libsolidity/SolidityNameAndTypeResolution.cpp b/test/libsolidity/SolidityNameAndTypeResolution.cpp index ab0f9c7b..882557fd 100644 --- a/test/libsolidity/SolidityNameAndTypeResolution.cpp +++ b/test/libsolidity/SolidityNameAndTypeResolution.cpp @@ -1124,6 +1124,17 @@ BOOST_AUTO_TEST_CASE(fallback_function_with_return_parameters) BOOST_CHECK(expectError(text) == Error::Type::TypeError); } +BOOST_AUTO_TEST_CASE(fallback_function_with_constant_modifier) +{ + char const* text = R"( + contract C { + uint x; + function() constant { x = 2; } + } + )"; + BOOST_CHECK(expectError(text) == Error::Type::TypeError); +} + BOOST_AUTO_TEST_CASE(fallback_function_twice) { char const* text = R"( |