diff options
| author | Roel Kluin <roel.kluin@gmail.com> | 2009-07-31 10:21:01 -0400 |
|---|---|---|
| committer | David Woodhouse <David.Woodhouse@intel.com> | 2009-09-04 04:40:17 -0400 |
| commit | c6f7e7beb9e6a64816f534a3a0dd0cfa4937f1fe (patch) | |
| tree | c0fed3deeb9862dffac83ac45814e70341317c2e /drivers/mtd/tests/mtd_oobtest.c | |
| parent | 269c0ee66367b11de9758ee64ea039843f0c7cad (diff) | |
mtd: tests: fix read buffer overflows
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>
Diffstat (limited to 'drivers/mtd/tests/mtd_oobtest.c')
| -rw-r--r-- | drivers/mtd/tests/mtd_oobtest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mtd/tests/mtd_oobtest.c b/drivers/mtd/tests/mtd_oobtest.c index a18e8d2f2557..5553cd4eab20 100644 --- a/drivers/mtd/tests/mtd_oobtest.c +++ b/drivers/mtd/tests/mtd_oobtest.c | |||
| @@ -512,7 +512,7 @@ static int __init mtd_oobtest_init(void) | |||
| 512 | goto out; | 512 | goto out; |
| 513 | 513 | ||
| 514 | addr0 = 0; | 514 | addr0 = 0; |
| 515 | for (i = 0; bbt[i] && i < ebcnt; ++i) | 515 | for (i = 0; i < ebcnt && bbt[i]; ++i) |
| 516 | addr0 += mtd->erasesize; | 516 | addr0 += mtd->erasesize; |
| 517 | 517 | ||
| 518 | /* Attempt to write off end of OOB */ | 518 | /* Attempt to write off end of OOB */ |