diff options
author | Patrick Georgi <patrick@georgi-clan.de> | 2014-09-29 20:23:29 +0200 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2014-10-18 14:45:24 +0200 |
commit | f0bd87153139e92fbcfa76b8fcab4558ce3cd52e (patch) | |
tree | 11980a46845485c8527dd6ae2b3eeda3a0d2df6a /src/mainboard/lippert | |
parent | 96990a285d775520c6279603371a1394bcdd8cd3 (diff) |
various AMD boards: fix buffer overflow
"AMD\t " isn't 8 characters long.
Change-Id: I47b2a39d7dca0201b7ee5dfd1f77e0714411257c
Found-by: Coverity Scan
Signed-off-by: Patrick Georgi <patrick@georgi-clan.de>
Reviewed-on: http://review.coreboot.org/6991
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-by: Jens Rottmann <JRottmann@LiPPERTembedded.de>
Diffstat (limited to 'src/mainboard/lippert')
-rw-r--r-- | src/mainboard/lippert/frontrunner-af/mptable.c | 2 | ||||
-rw-r--r-- | src/mainboard/lippert/toucan-af/mptable.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/mainboard/lippert/frontrunner-af/mptable.c b/src/mainboard/lippert/frontrunner-af/mptable.c index de573b8235..078601e56f 100644 --- a/src/mainboard/lippert/frontrunner-af/mptable.c +++ b/src/mainboard/lippert/frontrunner-af/mptable.c @@ -54,7 +54,7 @@ static void *smp_write_config_table(void *v) mc = (void *)(((char *)v) + SMP_FLOATING_TABLE_LEN); mptable_init(mc, LOCAL_APIC_ADDR); - memcpy(mc->mpc_oem, "AMD ", 8); + memcpy(mc->mpc_oem, "AMD ", 8); smp_write_processors(mc); diff --git a/src/mainboard/lippert/toucan-af/mptable.c b/src/mainboard/lippert/toucan-af/mptable.c index de573b8235..078601e56f 100644 --- a/src/mainboard/lippert/toucan-af/mptable.c +++ b/src/mainboard/lippert/toucan-af/mptable.c @@ -54,7 +54,7 @@ static void *smp_write_config_table(void *v) mc = (void *)(((char *)v) + SMP_FLOATING_TABLE_LEN); mptable_init(mc, LOCAL_APIC_ADDR); - memcpy(mc->mpc_oem, "AMD ", 8); + memcpy(mc->mpc_oem, "AMD ", 8); smp_write_processors(mc); |