[coreboot-gerrit] Change in coreboot[master]: lib/spd: respect spd memory part name override if present

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45459 )

Change subject: lib/spd: respect spd memory part name override if present
......................................................................

Patch Set 12:

(2 comments)

https://review.coreboot.org/c/coreboot/+/45459/12/src/lib/spd_bin.c 
File src/lib/spd_bin.c:

https://review.coreboot.org/c/coreboot/+/45459/12/src/lib/spd_bin.c@140 
PS12, Line 140: uint8_t spd[]
why is spd passed in when you're trying to define the part name that is out of band from the spd?

https://review.coreboot.org/c/coreboot/+/45459/12/src/lib/spd_bin.c@177 
PS12, Line 177: 		spd_name[name_len + 1] = 0;
This is actually a change. e.g. in ddr4 case this index is now DDR4_SPD_PART_LEN + 1 where it wasn't prior, and it is now an out of bound access.

This function needs the buffer size of spd_name to be correct.