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 (it's a string literal in both
call sites) to be less than 32 characters long.

Does anyone have any ideas to make Coverity shut up without making the
code unnecessarily ugly?

Thanks in advance,
Angel
_______________________________________________
coreboot mailing list -- coreboot@coreboot.org
To unsubscribe send an email to coreboot-le...@coreboot.org

Reply via email to