Merge pull request #642 from ethereum/fixLastOpIsUndefined

Check if vm trace opcodes are populated

Merge pull request #642 from ethereum/fixLastOpIsUndefined
ac571688 · yann300 · GitHub · 5ef18d9f · c5c4fa60 · ac571688
Unverified Commit ac571688 authored Dec 19, 2017 by yann300 Committed by GitHub Dec 19, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

web3VmProvider.js remix-lib/src/web3Provider/web3VmProvider.js +3 -1

No files found.
--- a/remix-lib/src/web3Provider/web3VmProvider.js
+++ b/remix-lib/src/web3Provider/web3VmProvider.js
@@ -97,7 +97,9 @@ web3VmProvider.prototype.txWillProcess = function (self, data) {

 web3VmProvider.prototype.txProcessed = function (self, data) {
  var lastOp = self.vmTraces[self.processingHash].structLogs[self.processingIndex - 1]
-  lastOp.error = lastOp.op !== 'RETURN' && lastOp.op !== 'STOP'
+  if (lastOp) {
+    lastOp.error = lastOp.op !== 'RETURN' && lastOp.op !== 'STOP' && lastOp.op !== 'SELFDESTRUCT'
+  }
  self.vmTraces[self.processingHash].gas = '0x' + data.gasUsed.toString(16)

  var logs = []