diff options
author | Rafał Miłecki <zajec5@gmail.com> | 2015-04-24 14:28:57 +0000 |
---|---|---|
committer | Rafał Miłecki <zajec5@gmail.com> | 2015-04-24 14:28:57 +0000 |
commit | 97a0e165a621c8b2d3b2e3e623ee1629166e66ed (patch) | |
tree | c1e8d402d36eea72dc1475670eafbd888cbcfc28 /package | |
parent | 84a3e668fc852afea25aadf8ee6f04a661fae280 (diff) | |
download | mtk-20170518-97a0e165a621c8b2d3b2e3e623ee1629166e66ed.zip mtk-20170518-97a0e165a621c8b2d3b2e3e623ee1629166e66ed.tar.gz mtk-20170518-97a0e165a621c8b2d3b2e3e623ee1629166e66ed.tar.bz2 |
nvram: fix regression in finding NVRAM beginning
The loop was giving up too early as it never expected NVRAM smaller
than 0x10000.
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
SVN-Revision: 45578
Diffstat (limited to 'package')
-rw-r--r-- | package/utils/nvram/src/nvram.c | 7 | ||||
-rw-r--r-- | package/utils/nvram/src/nvram.h | 1 |
2 files changed, 7 insertions, 1 deletions
diff --git a/package/utils/nvram/src/nvram.c b/package/utils/nvram/src/nvram.c index 0e1c726..c4bcb1d 100644 --- a/package/utils/nvram/src/nvram.c +++ b/package/utils/nvram/src/nvram.c @@ -364,7 +364,12 @@ nvram_handle_t * nvram_open(const char *file, int rdonly) if( mmap_area != MAP_FAILED ) { - for( i = 0; i <= ((nvram_part_size - NVRAM_SPACE) / sizeof(uint32_t)); i++ ) + /* + * Start looking for NVRAM_MAGIC at beginning of MTD + * partition. Stop if there is less than NVRAM_MIN_SPACE + * to check, that was the lowest used size. + */ + for( i = 0; i <= ((nvram_part_size - NVRAM_MIN_SPACE) / sizeof(uint32_t)); i++ ) { if( ((uint32_t *)mmap_area)[i] == NVRAM_MAGIC ) { diff --git a/package/utils/nvram/src/nvram.h b/package/utils/nvram/src/nvram.h index cd37650..215d816 100644 --- a/package/utils/nvram/src/nvram.h +++ b/package/utils/nvram/src/nvram.h @@ -113,6 +113,7 @@ char * nvram_find_staging(void); #define NVRAM_ROUNDUP(x, y) ((((x)+((y)-1))/(y))*(y)) /* NVRAM constants */ +#define NVRAM_MIN_SPACE 0x8000 #define NVRAM_SPACE 0x10000 #define NVRAM_MAGIC 0x48534C46 /* 'FLSH' */ #define NVRAM_VERSION 1 |