coreboot-kgpe-d16/payloads
Julius Werner d61350c403 libpayload: malloc: Change memcpy() to memmove() in realloc
Our realloc() works (somewhat suboptimally) by free()ing the existing
allocation and then reallocating it wherever it fits. If there was free
space before the old location, this means the new allocation may be
before the old one, and if the free space block is smaller than the old
allocation it may overlap. Thus, we should be moving memmove() instead
of memcpy() to move the block over.

This is not a problem in practice since all our existing memcpy()s are
simple iterate and copy front to back implementations which are safe for
overlaps when the destination is in front of the source. but it's still
the more correct thing to do (in case we ever change our memcpy()s to do
something more advanced or whatever).

Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: I35f77a94b7a72c01364ee7eecb5c3ff5ecde57f6
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40028
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
2020-04-03 19:56:27 +00:00
..
coreinfo payloads: Fix typos 2020-02-17 16:01:50 +00:00
external cros_ec: add chrome EC headers to include path 2020-04-01 09:19:48 +00:00
libpayload libpayload: malloc: Change memcpy() to memmove() in realloc 2020-04-03 19:56:27 +00:00
linuxcheck Makefile.inc, payloads: Enable -Wvla 2019-08-20 20:57:01 +00:00
nvramcui Makefile.inc, payloads: Enable -Wvla 2019-08-20 20:57:01 +00:00
Kconfig cbfs: allow uncompressed payloads 2020-02-22 22:38:27 +00:00
Makefile.inc payloads/external: Add yabits payload 2018-09-16 13:10:17 +00:00