logos-storage/logos-storage-nim
1
0
Fork 0
mirror of https://github.com/logos-storage/logos-storage-nim.git synced 2026-06-07 02:59:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix: test for correct verify root in slot builder.

Browse Source
This commit is contained in:
benbierens 2024-01-16 15:45:23 +01:00 committed by Dmitriy Ryajov
parent 9c1ea719d8
commit 964fe47ea2
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
codex/slots/builder/builder.nim
View File

@ -354,6 +354,14 @@ proc new*(
error "Failed to build slot roots tree", err = err.msg
return failure(err)
without expectedRoot =? manifest.verifyRoot.fromVerifyCid(), err:
error "Unable to convert manifest verifyRoot to hash", error = err.msg
return failure(err)
if verifyRoot =? tree.root:
if verifyRoot != expectedRoot:
return failure "Existing slots root doesn't match reconstructed root."
self.slotRoots = slotRoots
self.verifyTree = some tree

2
tests/codex/slots/testslotbuilder.nim
View File

@ -361,7 +361,7 @@ suite "Slot builder":
verifyManifest,
cellSize = cellSize).isErr
test "Should not build from verifiable manifest with slots root":
test "Should not build from verifiable manifest with invalid verify root":
let
slotBuilder = SlotsBuilder.new(
localStore,