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-2F2Czv4BOaCWWCy7my0P0...