[coreboot-gerrit] Change in coreboot[master]: [WIP]cpu/x86/smm: Use struct region to check overlapping sections

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63602 )

Change subject: [WIP]cpu/x86/smm: Use struct region to check overlapping sections
......................................................................

Patch Set 1:

(1 comment)

File src/cpu/x86/smm/smm_module_loader.c:

https://review.coreboot.org/c/coreboot/+/63602/comment/9a7ef304_8cd1691a 
PS1, Line 399: struct region_list *entry = list;
             : 	struct region_list *next_entry;
             : 	while (1) {
             : 		if (entry == NULL)
             : 			return;
             : 		next_entry = entry->next;
             : 		free(entry);
             : 		entry = next_entry;
             : 	}
won't work. Only the last allocated element is freed.