Jonathan Neuschäfer has posted comments on this change. ( https://review.coreboot.org/29045 )
Change subject: This patch fixes Klockwork issues due to the possibility of a NULL pointer being dereferenced ......................................................................
Patch Set 2: Code-Review+1
(2 comments)
https://review.coreboot.org/#/c/29045/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/29045/2//COMMIT_MSG@7 PS2, Line 7: This patch fixes Klockwork issues due to the possibility of a NULL pointer being dereferenced Please use a slightly shorter commit title, like "Fix potential NULL pointer dereferences reported by Klockwork"
https://review.coreboot.org/#/c/29045/2/src/arch/x86/acpigen.c File src/arch/x86/acpigen.c:
https://review.coreboot.org/#/c/29045/2/src/arch/x86/acpigen.c@289 PS2, Line 289: /* If we have an incoming NULL namepath Then we need to put a null Please use a comma (,) before "then", rather than spelling "then" in uppercase.