diff options
| author | Roel Kluin <roel.kluin@gmail.com> | 2009-09-18 12:51:50 -0700 |
|---|---|---|
| committer | David Woodhouse <David.Woodhouse@intel.com> | 2009-09-19 11:23:26 -0700 |
| commit | a57ca0466af5da83e379d636b8c01fd53b41e2c6 (patch) | |
| tree | c5f5d96fd9729aafe24aff8e6974bd6eea78a250 /drivers | |
| parent | ebf2e93036907fe2a7ddab942aa63d35f97f3b2b (diff) | |
| download | op-kernel-dev-a57ca0466af5da83e379d636b8c01fd53b41e2c6.zip op-kernel-dev-a57ca0466af5da83e379d636b8c01fd53b41e2c6.tar.gz | |
mtd: mtdpart: prevent a read from regions[-1]
If the erase region was found in the first iteration we read from
regions[-1]
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/mtd/mtdpart.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/mtd/mtdpart.c b/drivers/mtd/mtdpart.c index 349fcbe..a83cfa1 100644 --- a/drivers/mtd/mtdpart.c +++ b/drivers/mtd/mtdpart.c @@ -453,7 +453,8 @@ static struct mtd_part *add_one_partition(struct mtd_info *master, for (i = 0; i < max && regions[i].offset <= slave->offset; i++) ; /* The loop searched for the region _behind_ the first one */ - i--; + if (i > 0) + i--; /* Pick biggest erasesize */ for (; i < max && regions[i].offset < end; i++) { |
