Attention is currently required from: Aarya, Anastasia Klimchuk, Nikolai Artemiev.
Hello Aarya, Anastasia Klimchuk, Nikolai Artemiev, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/81548?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed: Verified+1 by build bot (Jenkins)
Change subject: erasure_layout: don't copy region buffers if they're null/zero-size ......................................................................
erasure_layout: don't copy region buffers if they're null/zero-size
memcpy() function expects 2nd parameter to be non-null. Make sure that the pointer is null before passing it to the function.
Also move allocations under if conditions to avoid allocating memory for a potentially 0 size.
Found-by: scan-build, clang v17.0.6 Signed-off-by: Alexander Goncharov chat@joursoir.net Change-Id: I99aad4119f9c11bea75e3419926cc833bc1f68c5 --- M erasure_layout.c 1 file changed, 30 insertions(+), 24 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/48/81548/2