[coreboot-gerrit] Change in coreboot[master]: vboot: use vboot persistent context

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36300 )

Change subject: vboot: use vboot persistent context
......................................................................

Patch Set 7:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36300/7/src/security/vboot/common.c 
File src/security/vboot/common.c:

https://review.coreboot.org/c/coreboot/+/36300/7/src/security/vboot/common.c... 
PS7, Line 49: memset(wd, 0, VB2_FIRMWARE_WORKBUF_RECOMMENDED_SIZE);
Is the responsibility to 0-initialize entire work buffer moved to vboot now? Or is it deemed not necessary?

https://review.coreboot.org/c/coreboot/+/36300/7/src/security/vboot/common.c... 
PS7, Line 139: 	memcpy(wd_cbmem, wd_preram, sizeof(struct vboot_working_data));
> Don't you need another […]

Yes, that is required.