From 1142b99a643a676b9adb7ada0b19aa1c5d8f8401 Mon Sep 17 00:00:00 2001 From: liana Date: Fri, 16 Jan 2015 12:55:49 +0100 Subject: [PATCH] Corrected "delete" for local variables i.e. set them to 0 Added test case --- ExpressionCompiler.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ExpressionCompiler.cpp b/ExpressionCompiler.cpp index c8b7f3508..4eed68ea4 100644 --- a/ExpressionCompiler.cpp +++ b/ExpressionCompiler.cpp @@ -810,9 +810,10 @@ void ExpressionCompiler::LValue::setToZero(Expression const& _expression) const if (stackDiff > 16) BOOST_THROW_EXCEPTION(CompilerError() << errinfo_sourceLocation(_expression.getLocation()) << errinfo_comment("Stack too deep.")); - else if (stackDiff > 0) - for (unsigned i = 0; i < m_size; ++i) - *m_context << u256(0) << eth::swapInstruction(m_size - i) << eth::Instruction::POP; + solAssert(stackDiff <= m_size, ""); + for (unsigned i = 0; i < m_size; ++i) + *m_context << u256(0) << eth::swapInstruction(stackDiff + (m_size - i) ) << eth::Instruction::POP; + break; } case LValue::STORAGE: