diff options
author | Nicholas Piggin <npiggin@gmail.com> | 2018-03-26 15:02:44 +1000 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-03-27 00:42:50 -0500 |
commit | a1c3dcca81cec2755c64b97305a045382b74d2aa (patch) | |
tree | 602e3a059aff3e5c53f476801b0ded2dfb25ae10 /core/nvram-format.c | |
parent | c32943bfc1e254176ecab564fdb4752403a48cab (diff) | |
download | skiboot-a1c3dcca81cec2755c64b97305a045382b74d2aa.zip skiboot-a1c3dcca81cec2755c64b97305a045382b74d2aa.tar.gz skiboot-a1c3dcca81cec2755c64b97305a045382b74d2aa.tar.bz2 |
nvram: run nvram_validate() after nvram_reformat()
nvram_reformat() sets nvram_valid = true, but it does not set
skiboot_part_hdr. Call nvram_validate() instead, which sets
everything up properly.
Reviewed-by: Oliver O'Halloran <oohall@gmail.com>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core/nvram-format.c')
-rw-r--r-- | core/nvram-format.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/core/nvram-format.c b/core/nvram-format.c index 3d030a3..42c5cbb 100644 --- a/core/nvram-format.c +++ b/core/nvram-format.c @@ -180,6 +180,7 @@ int nvram_check(void *nvram_image, const uint32_t nvram_size) } prlog(PR_INFO, "NVRAM: Layout appears sane\n"); + assert(skiboot_part_hdr); return 0; failed: return -1; @@ -234,6 +235,8 @@ const char *nvram_query(const char *key) if (!nvram_validate()) return NULL; + assert(skiboot_part_hdr); + part_end = (const char *) skiboot_part_hdr + be16_to_cpu(skiboot_part_hdr->len) * 16 - 1; |