Commit 7c0b2380 authored by Dan Carpenter's avatar Dan Carpenter Committed by Lee Jones

mfd: ab8500-debugfs: Move dereference after check for NULL

We dereference "desc" before check if it is NULL. I've shifted it
around so we check first before dereferencing.
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
parent d6e0a2dd
...@@ -1600,7 +1600,6 @@ static int ab8500_interrupts_print(struct seq_file *s, void *p) ...@@ -1600,7 +1600,6 @@ static int ab8500_interrupts_print(struct seq_file *s, void *p)
for (line = 0; line < num_interrupt_lines; line++) { for (line = 0; line < num_interrupt_lines; line++) {
struct irq_desc *desc = irq_to_desc(line + irq_first); struct irq_desc *desc = irq_to_desc(line + irq_first);
struct irqaction *action = desc->action;
seq_printf(s, "%3i: %6i %4i", line, seq_printf(s, "%3i: %6i %4i", line,
num_interrupts[line], num_interrupts[line],
...@@ -1608,7 +1607,9 @@ static int ab8500_interrupts_print(struct seq_file *s, void *p) ...@@ -1608,7 +1607,9 @@ static int ab8500_interrupts_print(struct seq_file *s, void *p)
if (desc && desc->name) if (desc && desc->name)
seq_printf(s, "-%-8s", desc->name); seq_printf(s, "-%-8s", desc->name);
if (action) { if (desc && desc->action) {
struct irqaction *action = desc->action;
seq_printf(s, " %s", action->name); seq_printf(s, " %s", action->name);
while ((action = action->next) != NULL) while ((action = action->next) != NULL)
seq_printf(s, ", %s", action->name); seq_printf(s, ", %s", action->name);
......
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