Skip to content
This repository has been archived by the owner on Aug 14, 2023. It is now read-only.

Protected Mode for verify #334

Merged
merged 19 commits into from Aug 16, 2021
Merged

Protected Mode for verify #334

merged 19 commits into from Aug 16, 2021

Conversation

YaronWittenstein
Copy link
Contributor

@YaronWittenstein YaronWittenstein commented Aug 10, 2021

This PR restricts storage access while running the svm_verify and svm_alloc functions.

  • When running svm_alloc there is no reason to have access to the storage of an Account.
  • When running svm_verify we won't it to executed logic to be stateless.
    More information under the Account Unification SMIP.

@YaronWittenstein YaronWittenstein self-assigned this Aug 10, 2021
@YaronWittenstein YaronWittenstein added AA Related to the Accounts Abstraction svm svm-core SVM core change v0.3 labels Aug 10, 2021
@YaronWittenstein YaronWittenstein added this to the Aug 2 Sprint milestone Aug 10, 2021
@YaronWittenstein YaronWittenstein linked an issue Aug 10, 2021 that may be closed by this pull request
@YaronWittenstein YaronWittenstein marked this pull request as ready for review August 16, 2021 10:12
Copy link
Contributor

@neysofu neysofu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@YaronWittenstein YaronWittenstein merged commit 8fb9855 into master Aug 16, 2021
@YaronWittenstein YaronWittenstein deleted the verify-protected-mode branch August 16, 2021 10:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
AA Related to the Accounts Abstraction svm svm-core SVM core change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AA: Protected Mode for verify
2 participants