[flashrom-gerrit] [M] Change in flashrom[main]: erasure_layout: don't copy region buffers if they're null/zero-size

[Alexander Goncharov (Code Review)]

Attention is currently required from: Aarya, Anastasia Klimchuk, Nikolai Artemiev.

Alexander Goncharov has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/81548?usp=email )

Change subject: erasure_layout: don't copy region buffers if they're null/zero-size
......................................................................

Patch Set 2:

(1 comment)

File erasure_layout.c:

https://review.coreboot.org/c/flashrom/+/81548/comment/0c1561f9_cb1bdc70 :
PS1, Line 263: 	old_start_buf = (uint8_t *)malloc(old_start - region_start);
             : 	if (!old_start_buf) {
             : 		msg_cerr("Not enough memory!\n");
             : 		ret = -1;
             : 		goto _end;
             : 	}
             : 	old_end_buf = (uint8_t *)malloc(region_end - old_end);
             : 	if (!old_end_buf) {
             : 		msg_cerr("Not enough memory!\n");
             : 		ret = -1;
             : 		goto _end;
             : 	}
> Thanks for the details, Anastasia! Yeah, I will dive deeper to see possible solutions and understand […]

Sorry it took longer than I expected. Please, Anastasia, check out the new patch set