Skip to content

dm vdo: additional metadata validation

Matthew Sakai dm-devel 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 0 repliesNew
  2. v12 patches 1 replySuperseded

Changes in v2 from the cover letter

1/2 Fixed the upper bound check to handle MAX_ZONES correctly. 1/2 Added a lower bound check for the region count.

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2dm vdo indexer: validate saved region and zone countsdrivers/md/dm-vdo/indexer/index-layout.c+8 -10 replies
2/2dm vdo slab-depot: validate old zone count on loaddrivers/md/dm-vdo/slab-depot.c+4 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH v2 0/2] dm vdo: additional metadata validation

We identified a few places where dm-vdo uses a value from storage without sanity-checking that its value makes sense. Adding additional checks can prevent additional metadata corruption in the unusual cases where these values get corrupted.

Changes from v1:
1/2 Fixed the upper bound check to handle MAX_ZONES correctly. 1/2 Added a lower bound check for the region count.

Matthew Sakai (2):
dm vdo indexer: validate saved region and zone counts dm vdo slab-depot: validate old zone count on load

 drivers/md/dm-vdo/indexer/index-layout.c | 9 ++++++++-
 drivers/md/dm-vdo/slab-depot.c           | 4 ++++
 2 files changed, 12 insertions(+), 1 deletion(-)
Signature · 1 line
2.48.1