Skip to content
This repository has been archived by the owner on May 13, 2022. It is now read-only.

Call to selfdestruct(..) stops the chain #542

Closed
8 tasks
marc4gov opened this issue Mar 28, 2017 · 4 comments · Fixed by #605
Closed
8 tasks

Call to selfdestruct(..) stops the chain #542

marc4gov opened this issue Mar 28, 2017 · 4 comments · Fixed by #605

Comments

@marc4gov
Copy link

Please include in your bug report:

  • eris-db version (docker image tag or branch if built from source)
    0.12.0
  • eris version (if applicable)
    0.12.0
  • go version (if applicable)
  • docker version (if applicable)
    1.13.0
  • operating system details (osx/windows/linux)
    osx
  • what you expected to happen
    chain keeps alive
  • what actually happened
    chain stops
  • steps to reproduction
    I'm making a call to Solidity selfdestruct via a cascading contract call, e.g.:

From a TaskManager contract:

    function removeTask(address manager, address taskAddress)
    returns (bool result) {
        Task t = Task(taskAddress);
        if (t.remove(manager)) {
            taskList.removeElement(t);
            TaskDeleted(t, "Task Deleted", block.number, block.timestamp);
            return true;
        } else {
            return false;
        }
    }

call from within the Task contract:

    function remove(address manager)
    onlyManager(manager)
    isEmpty(manager)
    returns (bool)
    {
        selfdestruct(owner);
        return true;
    }

It doesn't matter if I use the owner or the manager address for the refund

@silasdavis
Copy link
Contributor

Will need to reproduce and pull a trace to see what is going on here.

@zramsay
Copy link
Contributor

zramsay commented Apr 25, 2017

https://github.com/monax/cli/pull/1344 may be relevant

@silasdavis
Copy link
Contributor

Minimal-ish reproduction here: https://gist.github.com/silasdavis/7f35cb31a255c18d2fc70eed1fa38211

@silasdavis
Copy link
Contributor

Fixed on develop

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging a pull request may close this issue.

3 participants