[coreboot-gerrit] Change in ...coreboot[master]: libpayload/option table: Don't pad string entries with garbage

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31348 )

Change subject: libpayload/option table: Don't pad string entries with garbage
......................................................................

libpayload/option table: Don't pad string entries with garbage

set_option_with() expects a buffer of the exact size of the option.

Change-Id: I21332394f88cf2daa4f733a544627d6d3c6ef26c
Signed-off-by: Nico Huber 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31348
Tested-by: build bot (Jenkins) 
Reviewed-by: Patrick Georgi 
---
M payloads/libpayload/drivers/options.c
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Georgi: Looks good to me, approved

diff --git a/payloads/libpayload/drivers/options.c b/payloads/libpayload/drivers/options.c
index 70c2b17..2b0a42e 100644
--- a/payloads/libpayload/drivers/options.c
+++ b/payloads/libpayload/drivers/options.c
@@ -357,7 +357,11 @@
    		*(u64*)raw = strtoull(value, NULL, 0);
    		break;
    	case 's':
-			raw = strdup(value);
+			raw = malloc(cmos_entry->length);
+			if (!raw)
+				return 1;
+			memset(raw, 0x00, cmos_entry->length);
+			strncpy(raw, value, cmos_entry->length);
    		break;
    	case 'e':
    		cmos_enum = lookup_cmos_enum_by_label(option_table, cmos_entry->config_id, value);