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

Revert SELFDESTRUCT fix #636

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 1 addition & 10 deletions manager/burrow-mint/state/block_cache.go
Original file line number Diff line number Diff line change
Expand Up @@ -197,9 +197,6 @@ func (cache *BlockCache) Sync() {
curStorage = storage
}
if curAccRemoved {
// We should delete any storage for removed accounts in anticipation of
// those accounts being reaped from the cache
delete(cache.storages, storageKey)
continue
}
value, dirty := cache.storages[storageKey].unpack()
Expand Down Expand Up @@ -227,12 +224,8 @@ func (cache *BlockCache) Sync() {
if removed {
removed := cache.backend.RemoveAccount([]byte(addrStr))
if !removed {
sanity.PanicCrisis(fmt.Sprintf("Could not remove account to be removed: %X", addrStr))
sanity.PanicCrisis(fmt.Sprintf("Could not remove account to be removed: %X", acc.Address))
}
// Since attempting to delete an already deleted account leads to a
// PanicCrisis (above) we must make sure we remove the tombstones of
// removed accounts
delete(cache.accounts, addrStr)
} else {
if acc == nil {
continue
Expand Down Expand Up @@ -266,8 +259,6 @@ func (cache *BlockCache) Sync() {
if !removed {
sanity.PanicCrisis(fmt.Sprintf("Could not remove namereg entry to be removed: %s", nameStr))
}
// Remove the tombstone name reg object from the cache
delete(cache.names, nameStr)
} else {
if entry == nil {
continue
Expand Down
103 changes: 0 additions & 103 deletions manager/burrow-mint/state/block_cache_test.go

This file was deleted.