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

Corrupted btree entries should not result in panics #191

Closed
arkpar opened this issue Mar 17, 2023 · 1 comment · Fixed by #194
Closed

Corrupted btree entries should not result in panics #191

arkpar opened this issue Mar 17, 2023 · 1 comment · Fixed by #194
Assignees

Comments

@arkpar
Copy link
Member

arkpar commented Mar 17, 2023

When parsing value table entries all expected reads should be validated agains actual buffer size. I believe regular hash value table already have checks in place. Btree entry parsing however does not and should be fixed.

Example of such panic: #190

@cheme
Copy link
Collaborator

cheme commented Mar 17, 2023

Right, will check this part (sounds like a nice monday morning task :)

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

Successfully merging a pull request may close this issue.

2 participants