Skip to content
This repository was archived by the owner on Jun 6, 2023. It is now read-only.
This repository was archived by the owner on Jun 6, 2023. It is now read-only.

StoragePower: OnConsensusFault incorrectly zeroes out miner power #692

Closed
#791
Closed
StoragePower: OnConsensusFault incorrectly zeroes out miner power#692
#791
Assignees
aarshkshah1992
Labels
P1High priority, required for basic network functionality and growthbugSomething isn't workingcode-readyReady for implementation, with clear pathgood first issueGood for newcomers
Milestone
💙 Mainnet
@wadealexc

Description

@wadealexc
wadealexc
opened on Jul 16, 2020

st.AddToClaim parameter order is flipped:

specs-actors/actors/builtin/power/power_actor.go

Line 251 in 13c9322

err = st.AddToClaim(adt.AsStore(rt), minerAddr, claim.QualityAdjPower.Neg(), claim.RawBytePower.Neg())

This method either always fails, or may leave a Miner with a non-zeroed-out Claim.

Metadata

Metadata

Assignees

Labels

P1High priority, required for basic network functionality and growthbugSomething isn't workingcode-readyReady for implementation, with clear pathgood first issueGood for newcomers

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions