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

blob: store the first share of the Blob in the blob's proof #2289

Open
Tracked by #2287
vgonkivs opened this issue May 31, 2023 · 2 comments
Open
Tracked by #2287

blob: store the first share of the Blob in the blob's proof #2289

vgonkivs opened this issue May 31, 2023 · 2 comments
Assignees

Comments

@vgonkivs
Copy link
Member

vgonkivs commented May 31, 2023

To protect against the length attack we should store the first share of the blob in the blob's proof.
A length attack is an attack when a malicious node can give us invalid proof and commitment and it could be proved against the header's DataRoot correctly. It can happen when the proof will contain only part of the subtreeRoots.

@vgonkivs vgonkivs changed the title Store the first share of the Blob in the blob's proof. blob: store the first share of the Blob in the blob's proof May 31, 2023
@nashqueue
Copy link
Member

From the first share of the Blob you can get the Sequence Length. From the sequence Length, you can check the length of the blob. If i don't, i can create a correct inclusion proof for 3/4 of a blob that is actually 4 shares long. The malicious commitment would be PFB2
image

@vgonkivs
Copy link
Member Author

vgonkivs commented Aug 2, 2024

#3610 (comment)

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

No branches or pull requests

3 participants