aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorchriseth <chris@ethereum.org>2018-03-15 01:04:40 +0800
committerGitHub <noreply@github.com>2018-03-15 01:04:40 +0800
commitabc7a45230614454d6ffb78447c112b63f712642 (patch)
tree5a02cbc3ae1416ded068c3e5f21762478c79ab55
parent59658afa30f434b3a2501c72d4a033af550462b0 (diff)
parent834d63de2c4dc9c119862f8bf25b4f7c9f408d6e (diff)
downloaddexon-solidity-abc7a45230614454d6ffb78447c112b63f712642.tar.gz
dexon-solidity-abc7a45230614454d6ffb78447c112b63f712642.tar.zst
dexon-solidity-abc7a45230614454d6ffb78447c112b63f712642.zip
Merge pull request #3725 from ethereum/blockhashNoCall
Allow ``block.blockhash`` without it being called.
-rw-r--r--Changelog.md1
-rw-r--r--libsolidity/codegen/ExpressionCompiler.cpp3
-rw-r--r--test/libsolidity/SolidityEndToEndTest.cpp17
3 files changed, 21 insertions, 0 deletions
diff --git a/Changelog.md b/Changelog.md
index 1b1d86b1..d05ad4de 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -3,6 +3,7 @@
Features:
Bugfixes:
+ * Code Generator: Allow ``block.blockhash`` without being called.
* Code Generator: Properly skip unneeded storgae array cleanup when not reducing length.
* Code Generator: Bugfix in modifier lookup in libraries.
* Commandline interface: Support ``--evm-version constantinople`` properly.
diff --git a/libsolidity/codegen/ExpressionCompiler.cpp b/libsolidity/codegen/ExpressionCompiler.cpp
index 7162cb0d..f50628ff 100644
--- a/libsolidity/codegen/ExpressionCompiler.cpp
+++ b/libsolidity/codegen/ExpressionCompiler.cpp
@@ -1147,6 +1147,9 @@ bool ExpressionCompiler::visit(MemberAccess const& _memberAccess)
else if (member == "sig")
m_context << u256(0) << Instruction::CALLDATALOAD
<< (u256(0xffffffff) << (256 - 32)) << Instruction::AND;
+ else if (member == "blockhash")
+ {
+ }
else
solAssert(false, "Unknown magic member.");
break;
diff --git a/test/libsolidity/SolidityEndToEndTest.cpp b/test/libsolidity/SolidityEndToEndTest.cpp
index 282136f7..a080fe71 100644
--- a/test/libsolidity/SolidityEndToEndTest.cpp
+++ b/test/libsolidity/SolidityEndToEndTest.cpp
@@ -1788,6 +1788,23 @@ BOOST_AUTO_TEST_CASE(transfer_ether)
ABI_CHECK(callContractFunction("b(address,uint256)", oogRecipient, 10), encodeArgs());
}
+BOOST_AUTO_TEST_CASE(uncalled_blockhash)
+{
+ char const* code = R"(
+ contract C {
+ function f() public view returns (bytes32)
+ {
+ var x = block.blockhash;
+ return x(block.number - 1);
+ }
+ }
+ )";
+ compileAndRun(code, 0, "C");
+ bytes result = callContractFunction("f()");
+ BOOST_REQUIRE_EQUAL(result.size(), 32);
+ BOOST_CHECK(result[0] != 0 || result[1] != 0 || result[2] != 0);
+}
+
BOOST_AUTO_TEST_CASE(log0)
{
char const* sourceCode = R"(