Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: handle errors and non-existent addresses in selfdestruct #897

Merged
merged 1 commit into from
Dec 2, 2022

Conversation

Stebalien
Copy link
Member

In the EVM:

  1. Self destruct continues even if it sends funds into oblivion.
  2. Self destruct will auto-create a beneficiary.

This lets us punt filecoin-project/ref-fvm#736 to M2.2.

@Stebalien
Copy link
Member Author

Blocked on #896.

@Stebalien

This comment was marked as resolved.

@Stebalien Stebalien force-pushed the steb/fix-selfdestruct branch 2 times, most recently from 8fce85c to 36d55b0 Compare December 2, 2022 05:51
@codecov-commenter
Copy link

Codecov Report

Merging #897 (36d55b0) into steb/refactor-table (b76083c) will increase coverage by 0.06%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@                   Coverage Diff                   @@
##           steb/refactor-table     #897      +/-   ##
=======================================================
+ Coverage                86.89%   86.96%   +0.06%     
=======================================================
  Files                      127      127              
  Lines                    23714    23717       +3     
=======================================================
+ Hits                     20607    20625      +18     
+ Misses                    3107     3092      -15     
Impacted Files Coverage Δ
...tors/evm/src/interpreter/instructions/lifecycle.rs 91.66% <100.00%> (+0.23%) ⬆️
actors/miner/src/lib.rs 83.26% <0.00%> (+0.12%) ⬆️
actors/power/src/lib.rs 84.92% <0.00%> (+0.40%) ⬆️
test_vm/src/lib.rs 80.88% <0.00%> (+0.73%) ⬆️

@Stebalien Stebalien force-pushed the steb/refactor-table branch 2 times, most recently from d6448db to a441f44 Compare December 2, 2022 14:55
Base automatically changed from steb/refactor-table to next December 2, 2022 15:16
In the EVM:

1. Self destruct continues even if it sends funds into oblivion.
2. Self destruct will auto-create a beneficiary.

This lets us punt filecoin-project/ref-fvm#736
to M2.2.
@Stebalien Stebalien enabled auto-merge (squash) December 2, 2022 15:25
@Stebalien Stebalien merged commit 0cc262a into next Dec 2, 2022
@Stebalien Stebalien deleted the steb/fix-selfdestruct branch December 2, 2022 15:42
Stebalien added a commit that referenced this pull request Dec 2, 2022
In the EVM:

1. Self destruct continues even if it sends funds into oblivion.
2. Self destruct will auto-create a beneficiary.

This lets us punt filecoin-project/ref-fvm#736
to M2.2.
@maciejwitowski maciejwitowski added Topic: Ethereum JSON-RPC bug Something isn't working labels Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants