build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32290 )
Change subject: Klocwork: Fix the Null pointer derefernce found by klocwork ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/#/c/32290/1/src/lib/coreboot_table.c File src/lib/coreboot_table.c:
https://review.coreboot.org/#/c/32290/1/src/lib/coreboot_table.c@236 PS1, Line 236: if(wd) code indent should use tabs where possible
https://review.coreboot.org/#/c/32290/1/src/lib/coreboot_table.c@236 PS1, Line 236: if(wd) please, no spaces at the start of a line
https://review.coreboot.org/#/c/32290/1/src/lib/coreboot_table.c@236 PS1, Line 236: if(wd) that open brace { should be on the previous line
https://review.coreboot.org/#/c/32290/1/src/lib/coreboot_table.c@236 PS1, Line 236: if(wd) space required before the open parenthesis '('
https://review.coreboot.org/#/c/32290/1/src/lib/coreboot_table.c@237 PS1, Line 237: { trailing whitespace