Skip to content

Commit

Permalink
yaffs2: Remove block number check from summary verification
Browse files Browse the repository at this point in the history
The summary already has other verification. This one is not needed.

The check caused summaries to be ignored if they were not on the
numbered block. This caused problems when a summary was embedded in an
image and the image is written to a flash with bad blocks.

Signed-off-by: Charles Manning <[email protected]>
  • Loading branch information
cdhmanning authored and trini committed Jan 20, 2014
1 parent b5bbac1 commit 5528363
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion fs/yaffs2/yaffs_summary.c
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,6 @@ int yaffs_summary_read(struct yaffs_dev *dev,
if (result == YAFFS_OK) {
/* Verify header */
if (hdr.version != YAFFS_SUMMARY_VERSION ||
hdr.block != blk ||
hdr.seq != bi->seq_number ||
hdr.sum != yaffs_summary_sum(dev))
result = YAFFS_FAIL;
Expand Down

0 comments on commit 5528363

Please sign in to comment.