On 10/05/18 07:53, Rob Herring wrote: > On Thu, Oct 4, 2018 at 11:14 PM <frowand.list@xxxxxxxxx> wrote: >> >> From: Frank Rowand <frank.rowand@xxxxxxxx> >> >> Callers of of_irq_parse_one() blindly use the pointer args.np >> without checking whether of_irq_parse_one() had an error and >> thus did not set the value of args.np. Initialize args to >> zero so that using the format "%pOF" to show the value of >> args.np will show "(null)" when of_irq_parse_one() has an >> error and does not set args.np instead of trying to >> dereference a random value. >> >> Reported-by: Guenter Roeck <linux@xxxxxxxxxxxx> >> Signed-off-by: Frank Rowand <frank.rowand@xxxxxxxx> >> --- >> drivers/of/unittest.c | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) > > Does this need to be part of this series? I do not know if it could be independently applied before the rest of the series (I have not tested that). I included it in the series because the series has so many other changes to unittest.c. If you want me to break this out, I will remove it from this series and resend it after the rest of the series has been pulled to mainline (and rebase on the new mainline). This patch is not fixing a known failure case - the current test data does not trigger the problem. The recent patch from Guenter that you already accepted fixes the known failure case, so this patch is not urgent. The same is true about the other cases Guenter pointed out that this patch should fix. -Frank
