[coreboot] Re: New Defects reported by Coverity Scan for coreboot

2022-10-12 Thread Angel Pons
Hi all, On Tue, Oct 11, 2022 at 7:03 PM Angel Pons wrote: > > Hi Patrick, > > On Tue, Oct 11, 2022 at 6:43 PM Patrick Georgi wrote: > > > > "Angel Pons" schrieb: > > > > > We made the patches that made Coverity angry about this `format_pn()` > > > function. However, this is not an actual bug: t

[coreboot] Re: New Defects reported by Coverity Scan for coreboot

2022-10-11 Thread Angel Pons
Hi Patrick, On Tue, Oct 11, 2022 at 6:43 PM Patrick Georgi wrote: > > "Angel Pons" schrieb: > > > We made the patches that made Coverity angry about this `format_pn()` > > function. However, this is not an actual bug: the > > `eeprom_read_serial()` function returns a buffer that is at most 32 >

[coreboot] Re: New Defects reported by Coverity Scan for coreboot

2022-10-11 Thread Patrick Georgi via coreboot
"Angel Pons" schrieb: > We made the patches that made Coverity angry about this `format_pn()` > function. However, this is not an actual bug: the > `eeprom_read_serial()` function returns a buffer that is at most 32 > (`HERMES_SN_PN_LENGTH`) characters long, and the length of the > `prefix` strin

[coreboot] Re: New Defects reported by Coverity Scan for coreboot

2022-10-11 Thread Angel Pons
Hi list, We made the patches that made Coverity angry about this `format_pn()` function. However, this is not an actual bug: the `eeprom_read_serial()` function returns a buffer that is at most 32 (`HERMES_SN_PN_LENGTH`) characters long, and the length of the `prefix` string is known at build-time