Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

How to catch malicios memory manipulation of ND-section: add check before *and* after program #182

Open
Sword-Smith opened this issue Sep 9, 2024 · 0 comments
Assignees

Comments

@Sword-Smith
Copy link
Member

As mentioned in #173 I've had some concern whether size indicators can be abused maliciously. The VerifyNdSiIntegrity goes a long way to addressing that concern.

A further idea could be to run that snippet both before and after the consensus program. That way it will be very difficult to mutate the ND-part of memory, as that would have to be done in a way that still preserves the size-indicator integrity.

With these two checks, and some more eyeballs on the final product, I feel confident that we can, at least with respect to the assembler code, produce consensus programs that are sound.

@Sword-Smith Sword-Smith self-assigned this Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant