diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2010-07-28 23:45:03 -0400 |
|---|---|---|
| committer | Wolfgang Denk <wd@denx.de> | 2010-08-09 01:03:54 +0200 |
| commit | a4e8d9f5f957c7e3972ca9d25f17de5f8acd8244 (patch) | |
| tree | 2f01e36b14fe7688b95f64b0e5f7ad14c61bace4 /common/flash.c | |
| parent | 9ed4a9582ff61225d46241a1c99795549722503c (diff) | |
flash_protect: check for NULL flash info
If a flash is unable to be detected, and then someone calls flash_protect
on it (like the common code does in flash_init), the flash_protect logic
will dereference a NULL pointer.
Since flash_protect already does sanity checking on the info structs, add
a NULL pointer check in there.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Diffstat (limited to 'common/flash.c')
| -rw-r--r-- | common/flash.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/common/flash.c b/common/flash.c index eb4b2f5ff1c..683978e473c 100644 --- a/common/flash.c +++ b/common/flash.c @@ -43,15 +43,18 @@ extern flash_info_t flash_info[]; /* info for FLASH chips */ void flash_protect (int flag, ulong from, ulong to, flash_info_t *info) { - ulong b_end = info->start[0] + info->size - 1; /* bank end address */ - short s_end = info->sector_count - 1; /* index of last sector */ + ulong b_end; + short s_end; int i; /* Do nothing if input data is bad. */ - if (info->sector_count == 0 || info->size == 0 || to < from) { + if (!info || info->sector_count == 0 || info->size == 0 || to < from) { return; } + s_end = info->sector_count - 1; /* index of last sector */ + b_end = info->start[0] + info->size - 1; /* bank end address */ + debug ("flash_protect %s: from 0x%08lX to 0x%08lX\n", (flag & FLAG_PROTECT_SET) ? "ON" : (flag & FLAG_PROTECT_CLEAR) ? "OFF" : "???", |