Skip to content

Fix/rollback revive#8

Open
davd-gzl wants to merge 5 commits intosamouraiworld:masterfrom
davd-gzl:fix/rollback-revive
Open

Fix/rollback revive#8
davd-gzl wants to merge 5 commits intosamouraiworld:masterfrom
davd-gzl:fix/rollback-revive

Conversation

@davd-gzl
Copy link
Copy Markdown

@davd-gzl davd-gzl commented Dec 9, 2025

Fix: gnolang#4782
Fix: gnolang#2043

WIP, this is a poc

There's 3 differents situations:

  • panic after crossing -> rollback already implemented in op_call.go:L357 thanks to store cache system
  • panic in same context -> copy the cache, and revert it (hence the cache deepCopy implementation)
  • no panic -> We've to panic in op_call.go

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lack of transactional isolation in revive function

1 participant