You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jun 6, 2023. It is now read-only.
builtin.RequireNoErr(rt, err, exitcode.ErrIllegalState, "failed to load state")
Market.AddBalance creates a marketStateMutation with WritePermission for st.LockedTable. As only st.EscrowTable is read from / written to, this permission is unnecessary and may cause extraneous state writes.
Recommendation
Remove the permission
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
specs-actors/actors/builtin/market/market_actor.go
Lines 130 to 133 in a9f9f95
Market.AddBalance
creates amarketStateMutation
withWritePermission
forst.LockedTable
. As onlyst.EscrowTable
is read from / written to, this permission is unnecessary and may cause extraneous state writes.Recommendation
Remove the permission
The text was updated successfully, but these errors were encountered: