Attention is currently required from: Jan Dabros.
1 comment:
File tests/lib/bootmem-test.c:
Patch Set #11, Line 232: lb_mem = malloc(sizeof(*lb_mem) + 10 * sizeof(struct lb_memory_range));
Found by Coverity: you should initialize lb_mem->size (and ideally check that the function modified […]
Size (and tag) initialization is missing, you are right. But size is present in assert in test_bootmem_write_mem_table(). I think, that it is not necessary to add assert here as it will be checked in mentioned test. I just have to include initial size in assertion :)
Please look at: https://review.coreboot.org/c/coreboot/+/52263
To view, visit change 43510. To unsubscribe, or for help writing mail filters, visit settings.