build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33625 )
Change subject: Fix NULL dereference issues found by Klocwork ......................................................................
Patch Set 6:
(18 comments)
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... File payloads/libpayload/curses/form/fty_alnum.c:
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 166: if (bp == NULL) please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 166: if (bp == NULL) suspect code indent for conditional statements (2, 4)
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 167: return result; please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... File payloads/libpayload/curses/form/fty_alpha.c:
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 166: if (bp == NULL) please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 166: if (bp == NULL) suspect code indent for conditional statements (2, 4)
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 167: return result; please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... File payloads/libpayload/curses/form/fty_enum.c:
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 296: if (bp == NULL) please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 296: if (bp == NULL) suspect code indent for conditional statements (2, 4)
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 297: return FALSE; please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 353: if (bp == NULL) please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 353: if (bp == NULL) suspect code indent for conditional statements (2, 4)
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 354: return FALSE; please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 394: if (bp == NULL) please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 394: if (bp == NULL) suspect code indent for conditional statements (2, 4)
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 395: return FALSE; please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... File payloads/libpayload/curses/form/fty_int.c:
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 175: if (bp == NULL) please, no spaces at the start of a line
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 175: if (bp == NULL) suspect code indent for conditional statements (2, 4)
https://review.coreboot.org/#/c/33625/6/payloads/libpayload/curses/form/fty_... PS6, Line 176: return result; please, no spaces at the start of a line