build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32083 )
Change subject: payload/libpayload: This is a fix for potential NULL pointer dereferences ......................................................................
Patch Set 6:
(5 comments)
https://review.coreboot.org/#/c/32083/6/payloads/libpayload/drivers/usb/ehci... File payloads/libpayload/drivers/usb/ehci.c:
https://review.coreboot.org/#/c/32083/6/payloads/libpayload/drivers/usb/ehci... PS6, Line 647: if (!next_td) trailing whitespace
https://review.coreboot.org/#/c/32083/6/payloads/libpayload/drivers/usb/ehci... PS6, Line 648: fatal("Not enough DMA memory to create #reqcount TD.\n"); line over 80 characters
https://review.coreboot.org/#/c/32083/6/payloads/libpayload/drivers/usb/ehci... PS6, Line 833: if (!EHCI_INST(controller)->dummy_qh) trailing whitespace
https://review.coreboot.org/#/c/32083/6/payloads/libpayload/drivers/usb/ohci... File payloads/libpayload/drivers/usb/ohci.c:
https://review.coreboot.org/#/c/32083/6/payloads/libpayload/drivers/usb/ohci... PS6, Line 577: fatal("Not enough DMA memory for OHCI addition TD in bulk transfer.\n"); line over 80 characters
https://review.coreboot.org/#/c/32083/6/payloads/libpayload/drivers/usb/ohci... PS6, Line 703: fatal("Not enough memory to add transfer descriptor.\n"); line over 80 characters