aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorchriseth <chris@ethereum.org>2016-12-03 08:08:45 +0800
committerGitHub <noreply@github.com>2016-12-03 08:08:45 +0800
commit9be2fb12bd4ff8b4109db67a236642a9d3b2fc15 (patch)
treeb914f7c105b618c48e6da10a46cc7572fcd44203
parent3a01a87afe3468421f31aa5097796dcc88e37e26 (diff)
parent86d54c02cdebba32a6a8fba1d74bad17667d46cd (diff)
downloaddexon-solidity-9be2fb12bd4ff8b4109db67a236642a9d3b2fc15.tar.gz
dexon-solidity-9be2fb12bd4ff8b4109db67a236642a9d3b2fc15.tar.zst
dexon-solidity-9be2fb12bd4ff8b4109db67a236642a9d3b2fc15.zip
Merge pull request #1475 from ethereum/fail-on-identity-fail
Throw if calling the identity precompile (memoryCopy) failed
-rw-r--r--Changelog.md1
-rw-r--r--libsolidity/codegen/CompilerUtils.cpp3
2 files changed, 3 insertions, 1 deletions
diff --git a/Changelog.md b/Changelog.md
index 1058fdf5..58cf1484 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -5,6 +5,7 @@ Features:
* Optimizer: Some dead code elimination.
Bugfixes:
+ * Code generator: throw if calling the identity precompile failed during memory (array) copying.
* Type checker: string literals that are not valid UTF-8 cannot be converted to string type
### 0.4.6 (2016-11-22)
diff --git a/libsolidity/codegen/CompilerUtils.cpp b/libsolidity/codegen/CompilerUtils.cpp
index d5361ac6..a8299626 100644
--- a/libsolidity/codegen/CompilerUtils.cpp
+++ b/libsolidity/codegen/CompilerUtils.cpp
@@ -312,7 +312,8 @@ void CompilerUtils::memoryCopy()
m_context << Instruction::DIV << u256(c_identityWordGas) << Instruction::MUL;
m_context << u256(c_identityGas) << Instruction::ADD;
m_context << Instruction::CALL;
- m_context << Instruction::POP; // ignore return value
+ m_context << Instruction::ISZERO;
+ m_context.appendConditionalJumpTo(m_context.errorTag());
}
void CompilerUtils::splitExternalFunctionType(bool _leftAligned)