Hi, Please find the latest report on new defect(s) introduced to coreboot found with Coverity Scan.
2 new defect(s) introduced to coreboot found with Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 2 of 2 defect(s) ** CID 1458054: Memory - corruptions (OVERRUN) /src/mainboard/google/dedede/variants/baseboard/ramstage.c: 41 in update_lte_device() ________________________________________________________________________________________________________ *** CID 1458054: Memory - corruptions (OVERRUN) /src/mainboard/google/dedede/variants/baseboard/ramstage.c: 41 in update_lte_device() 35 config->type = UPC_TYPE_INTERNAL; 36 if (port->path.usb.port_type == LTE_USB_PORT_TYPE) { 37 config->has_power_resource = 1; 38 memcpy(&config->reset_gpio, <e_reset_gpio, 39 sizeof(config->reset_gpio)); 40 config->reset_off_delay_ms = 20; >>> CID 1458054: Memory - corruptions (OVERRUN) >>> Overrunning buffer pointed to by "<e_enable_gpio" of 4 bytes by >>> passing it to a function which accesses it at byte offset 71 using argument >>> "72UL". [Note: The source code implementation of the function has been >>> overridden by a builtin model.] 41 memcpy(&config->enable_gpio, <e_enable_gpio, 42 sizeof(config->enable_gpio)); 43 config->enable_delay_ms = 20; 44 } 45 } 46 } 47 } ** CID 1458053: Memory - corruptions (OVERRUN) /src/mainboard/google/dedede/variants/baseboard/ramstage.c: 38 in update_lte_device() ________________________________________________________________________________________________________ *** CID 1458053: Memory - corruptions (OVERRUN) /src/mainboard/google/dedede/variants/baseboard/ramstage.c: 38 in update_lte_device() 32 fw_config_probe(FW_CONFIG(DB_PORTS, DB_PORTS_1A_HDMI_LTE))) { 33 34 config = port->chip_info; 35 config->type = UPC_TYPE_INTERNAL; 36 if (port->path.usb.port_type == LTE_USB_PORT_TYPE) { 37 config->has_power_resource = 1; >>> CID 1458053: Memory - corruptions (OVERRUN) >>> Overrunning buffer pointed to by "<e_reset_gpio" of 4 bytes by >>> passing it to a function which accesses it at byte offset 71 using argument >>> "72UL". [Note: The source code implementation of the function has been >>> overridden by a builtin model.] 38 memcpy(&config->reset_gpio, <e_reset_gpio, 39 sizeof(config->reset_gpio)); 40 config->reset_off_delay_ms = 20; 41 memcpy(&config->enable_gpio, <e_enable_gpio, 42 sizeof(config->enable_gpio)); 43 config->enable_delay_ms = 20; ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50yq2SfQfrHt3Prsn4qSLrYIrajINpiFX8l0vrlNSf8iCrS27qY0Cr0DkycwNUgGZJj8-3DWlVI_L-2FDzr14mnrsJO5b1wX1hp9b1MAQygl7x-2B74RAaH2cn3HiNC72a8zBdwg3-2FNS2rmv0RChO20VuIitqWAd3nzF2R4EOmRURa-2BimCsfyoKyqyCXlG8LrKARdDt9UkghPuw4Wsm1Qa75zKP0SLMtPJwb9CA8a7k8-2FcFb3L0yi86vC1bu12nd06l-2F6NAo6IZRX5k93RQ7pnR7G-2FkLnfbMrTX-2BtfZgKV4-2F-2F9nhvgkLHFjy5Bc-3D _______________________________________________ coreboot mailing list -- coreboot@coreboot.org To unsubscribe send an email to coreboot-le...@coreboot.org