mtd: tests: fix read buffer overflows
authorRoel Kluin <roel.kluin@gmail.com>
Fri, 31 Jul 2009 14:21:01 +0000 (16:21 +0200)
committerDavid Woodhouse <David.Woodhouse@intel.com>
Fri, 4 Sep 2009 08:40:17 +0000 (09:40 +0100)
Check whether index is within bounds before testing the element.

Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
drivers/mtd/tests/mtd_oobtest.c
drivers/mtd/tests/mtd_pagetest.c

index a18e8d2f255765db492c8b1797dd99c2eaf54ece..5553cd4eab2083d80824f49e95423e1e2507a705 100644 (file)
@@ -512,7 +512,7 @@ static int __init mtd_oobtest_init(void)
                goto out;
 
        addr0 = 0;
-       for (i = 0; bbt[i] && i < ebcnt; ++i)
+       for (i = 0; i < ebcnt && bbt[i]; ++i)
                addr0 += mtd->erasesize;
 
        /* Attempt to write off end of OOB */
index 9648818b9e2c3857feb9f1a2e57a159ef073b94a..103cac480feec2fbf622db0d4d65694cc611547a 100644 (file)
@@ -116,11 +116,11 @@ static int verify_eraseblock(int ebnum)
        loff_t addr = ebnum * mtd->erasesize;
 
        addr0 = 0;
-       for (i = 0; bbt[i] && i < ebcnt; ++i)
+       for (i = 0; i < ebcnt && bbt[i]; ++i)
                addr0 += mtd->erasesize;
 
        addrn = mtd->size;
-       for (i = 0; bbt[ebcnt - i - 1] && i < ebcnt; ++i)
+       for (i = 0; i < ebcnt && bbt[ebcnt - i - 1]; ++i)
                addrn -= mtd->erasesize;
 
        set_random_data(writebuf, mtd->erasesize);
@@ -219,11 +219,11 @@ static int crosstest(void)
        memset(pp1, 0, pgsize * 4);
 
        addr0 = 0;
-       for (i = 0; bbt[i] && i < ebcnt; ++i)
+       for (i = 0; i < ebcnt && bbt[i]; ++i)
                addr0 += mtd->erasesize;
 
        addrn = mtd->size;
-       for (i = 0; bbt[ebcnt - i - 1] && i < ebcnt; ++i)
+       for (i = 0; i < ebcnt && bbt[ebcnt - i - 1]; ++i)
                addrn -= mtd->erasesize;
 
        /* Read 2nd-to-last page to pp1 */
@@ -317,7 +317,7 @@ static int erasecrosstest(void)
 
        ebnum = 0;
        addr0 = 0;
-       for (i = 0; bbt[i] && i < ebcnt; ++i) {
+       for (i = 0; i < ebcnt && bbt[i]; ++i) {
                addr0 += mtd->erasesize;
                ebnum += 1;
        }
@@ -413,7 +413,7 @@ static int erasetest(void)
 
        ebnum = 0;
        addr0 = 0;
-       for (i = 0; bbt[i] && i < ebcnt; ++i) {
+       for (i = 0; i < ebcnt && bbt[i]; ++i) {
                addr0 += mtd->erasesize;
                ebnum += 1;
        }