diff options
author | Rod Whitby <rod@whitby.id.au> | 2007-02-10 04:26:48 -0500 |
---|---|---|
committer | David Woodhouse <dwmw2@infradead.org> | 2007-02-10 04:26:48 -0500 |
commit | 11192146e57bc8e58865e7d9c2497f66a4f7e6e7 (patch) | |
tree | a4ca503fce35895f4031b11b92be45f1d35c2f15 | |
parent | a2593247d747954cd12c32da8c5a3aecb9cd19a3 (diff) |
[MTD] Fix RedBoot partition parsing regression harder.
Correct the location of the recalculation of the FIS directory size,
and also add the same recalculation for the byte-swapped case.
Signed-off-by: Rod Whitby <rod@whitby.id.au>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
-rw-r--r-- | drivers/mtd/redboot.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/mtd/redboot.c b/drivers/mtd/redboot.c index 5a31f490368e..a459ffa1e676 100644 --- a/drivers/mtd/redboot.c +++ b/drivers/mtd/redboot.c | |||
@@ -96,6 +96,8 @@ static int parse_redboot_partitions(struct mtd_info *master, | |||
96 | */ | 96 | */ |
97 | if (swab32(buf[i].size) == master->erasesize) { | 97 | if (swab32(buf[i].size) == master->erasesize) { |
98 | int j; | 98 | int j; |
99 | /* Update numslots based on actual FIS directory size */ | ||
100 | numslots = swab32(buf[i].size) / sizeof (struct fis_image_desc); | ||
99 | for (j = 0; j < numslots; ++j) { | 101 | for (j = 0; j < numslots; ++j) { |
100 | 102 | ||
101 | /* A single 0xff denotes a deleted entry. | 103 | /* A single 0xff denotes a deleted entry. |
@@ -120,11 +122,11 @@ static int parse_redboot_partitions(struct mtd_info *master, | |||
120 | swab32s(&buf[j].desc_cksum); | 122 | swab32s(&buf[j].desc_cksum); |
121 | swab32s(&buf[j].file_cksum); | 123 | swab32s(&buf[j].file_cksum); |
122 | } | 124 | } |
125 | } else { | ||
126 | /* Update numslots based on actual FIS directory size */ | ||
127 | numslots = buf[i].size / sizeof(struct fis_image_desc); | ||
123 | } | 128 | } |
124 | break; | 129 | break; |
125 | } else if (buf[i].size != -1) { | ||
126 | /* re-calculate of real numslots */ | ||
127 | numslots = buf[i].size / sizeof(struct fis_image_desc); | ||
128 | } | 130 | } |
129 | } | 131 | } |
130 | if (i == numslots) { | 132 | if (i == numslots) { |