filecoin-project / filecoin-project/builtin-actors

Update then burn in verifreg's universal_receiver_hook

Open
#1,219 0 comments 0 reactions 0 assignees View on GitHub
cleanup good first issue P2
Dominant language
Rust
Stars
96
Forks
94
Avg merge
6d 17h
Merged PRs (30d)
3

Description

This _should_ be safe right now as `burn` calls a trusted actor, but it _could_ become an issue if we're not careful. Ideally we'd:

1. Update the state in a transaction.
2. Burn, rolling back the state update on failure.

See: https://github.com/filecoin-project/builtin-actors/blob/05bc73f1f511e7e1e25d3a0ac3ddf34ce6f9f745/actors/verifreg/src/lib.rs#L617-L703

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.