Commit 7a2eab0d authored by Tony Breeds's avatar Tony Breeds Committed by Paul Mackerras

powerpc: Protect against NULL pointer deref in phyp-dump code

print_dump_header() will be called at least once with a NULL pointer in
a normal boot sequence.  If DEBUG is defined then we will dereference
the pointer and crash.  Add a quick fix to exit early in the NULL pointer
case.
Signed-off-by: default avatarTony Breeds <tony@bakeyournoodle.com>
Acked-by: default avatarManish Ahuja <mahujam@gmail.com>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent 8168b540
...@@ -130,6 +130,9 @@ static unsigned long init_dump_header(struct phyp_dump_header *ph) ...@@ -130,6 +130,9 @@ static unsigned long init_dump_header(struct phyp_dump_header *ph)
static void print_dump_header(const struct phyp_dump_header *ph) static void print_dump_header(const struct phyp_dump_header *ph)
{ {
#ifdef DEBUG #ifdef DEBUG
if (ph == NULL)
return;
printk(KERN_INFO "dump header:\n"); printk(KERN_INFO "dump header:\n");
/* setup some ph->sections required */ /* setup some ph->sections required */
printk(KERN_INFO "version = %d\n", ph->version); printk(KERN_INFO "version = %d\n", ph->version);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment