summaryrefslogtreecommitdiff
path: root/common/flash.c
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2010-07-28 23:45:03 -0400
committerWolfgang Denk <wd@denx.de>2010-08-09 01:03:54 +0200
commita4e8d9f5f957c7e3972ca9d25f17de5f8acd8244 (patch)
tree2f01e36b14fe7688b95f64b0e5f7ad14c61bace4 /common/flash.c
parent9ed4a9582ff61225d46241a1c99795549722503c (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.c9
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" : "???",