From 0b6a26f8544f632b557681e26ebc2b395bf03b20 Mon Sep 17 00:00:00 2001 From: Alex Beregszaszi Date: Wed, 8 Aug 2018 22:54:33 +0100 Subject: Calculate the dataGas correctly in the constant optimiser This may cause a wrong decision about cost (and as a result choosing the least efficient code), but will not cause any miscompilation or invalid output. --- libevmasm/ConstantOptimiser.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'libevmasm/ConstantOptimiser.cpp') diff --git a/libevmasm/ConstantOptimiser.cpp b/libevmasm/ConstantOptimiser.cpp index d0b6843c..07ece12c 100644 --- a/libevmasm/ConstantOptimiser.cpp +++ b/libevmasm/ConstantOptimiser.cpp @@ -93,6 +93,7 @@ bigint ConstantOptimisationMethod::simpleRunGas(AssemblyItems const& _items) bigint ConstantOptimisationMethod::dataGas(bytes const& _data) const { + assertThrow(_data.size() > 0, OptimizerException, "Empty bytecode generated."); if (m_params.isCreation) { bigint gas; @@ -101,7 +102,7 @@ bigint ConstantOptimisationMethod::dataGas(bytes const& _data) const return gas; } else - return GasCosts::createDataGas * dataSize(); + return GasCosts::createDataGas * _data.size(); } size_t ConstantOptimisationMethod::bytesRequired(AssemblyItems const& _items) -- cgit v1.2.3