[MTD] Fix RedBoot partition parsing regression harder.
authorRod Whitby <rod@whitby.id.au>
Sat, 10 Feb 2007 09:26:48 +0000 (09:26 +0000)
committerDavid Woodhouse <dwmw2@infradead.org>
Sat, 10 Feb 2007 09:26:48 +0000 (09:26 +0000)
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>
drivers/mtd/redboot.c

index 5a31f490368ebb1977397a7510f9032543d972cd..a459ffa1e6766a1a1f414255573d8422c911e814 100644 (file)
@@ -96,6 +96,8 @@ static int parse_redboot_partitions(struct mtd_info *master,
                         */
                        if (swab32(buf[i].size) == master->erasesize) {
                                int j;
+                               /* Update numslots based on actual FIS directory size */
+                               numslots = swab32(buf[i].size) / sizeof (struct fis_image_desc);
                                for (j = 0; j < numslots; ++j) {
 
                                        /* A single 0xff denotes a deleted entry.
@@ -120,11 +122,11 @@ static int parse_redboot_partitions(struct mtd_info *master,
                                        swab32s(&buf[j].desc_cksum);
                                        swab32s(&buf[j].file_cksum);
                                }
+                       } else {
+                               /* Update numslots based on actual FIS directory size */
+                               numslots = buf[i].size / sizeof(struct fis_image_desc);
                        }
                        break;
-               } else if (buf[i].size != -1) {
-                       /* re-calculate of real numslots */
-                       numslots = buf[i].size / sizeof(struct fis_image_desc);
                }
        }
        if (i == numslots) {