paritytech / parity-db

Experimental blockchain database
Apache License 2.0
263 stars 59 forks source link

Corrupted btree entries should not result in panics #191

Closed arkpar closed 1 year ago

arkpar commented 1 year ago

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: https://github.com/paritytech/parity-db/issues/190

cheme commented 1 year ago

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