mirrors/linux-stable
1
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git synced 2024-11-01 00:48:50 +00:00
Code Issues Releases Wiki Activity

x86/microcode/AMD: Check microcode container data in the late loader

Browse source 
Convert the late loading path to use the newly introduced microcode
container data checking functions as it was previously done for the
early loader.

[ bp: Keep header length addition in install_equiv_cpu_table() and rediff. ]

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: x86@kernel.org
Link: https://lkml.kernel.org/r/20181107170218.7596-14-bp@alien8.de
This commit is contained in:
Maciej S. Szmigiero 2018-09-13 08:49:22 +02:00 committed by Borislav Petkov
parent 72dc571a3a
commit 38673f623d
1 changed files with 17 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

38
arch/x86/kernel/cpu/microcode/amd.c
View file

@ -706,28 +706,27 @@ static enum ucode_state apply_microcode_amd(int cpu)
return ret;
}
static int install_equiv_cpu_table(const u8 *buf)
static size_t install_equiv_cpu_table(const u8 *buf, size_t buf_size)
{
unsigned int *ibuf = (unsigned int *)buf;
unsigned int type = ibuf[1];
unsigned int size = ibuf[2];
u32 equiv_tbl_len;
const u32 *hdr;
if (type != UCODE_EQUIV_CPU_TABLE_TYPE || !size) {
pr_err("empty section/"
"invalid type field in container file section header\n");
return -EINVAL;
}
if (!verify_equivalence_table(buf, buf_size, false))
return 0;
equiv_cpu_table = vmalloc(size);
hdr = (const u32 *)buf;
equiv_tbl_len = hdr[2];
equiv_cpu_table = vmalloc(equiv_tbl_len);
if (!equiv_cpu_table) {
pr_err("failed to allocate equivalent CPU table\n");
return -ENOMEM;
return 0;
}
memcpy(equiv_cpu_table, buf + CONTAINER_HDR_SZ, size);
memcpy(equiv_cpu_table, buf + CONTAINER_HDR_SZ, equiv_tbl_len);
/* add header length */
return size + CONTAINER_HDR_SZ;
return equiv_tbl_len + CONTAINER_HDR_SZ;
}
static void free_equiv_cpu_table(void)
@ -794,13 +793,12 @@ static enum ucode_state __load_microcode_amd(u8 family, const u8 *data,
size_t size)
{
u8 *fw = (u8 *)data;
int offset;
size_t offset;
offset = install_equiv_cpu_table(data);
if (offset < 0) {
pr_err("failed to create equivalent cpu table\n");
offset = install_equiv_cpu_table(data, size);
if (!offset)
return UCODE_ERROR;
}
fw += offset;
size -= offset;
@ -898,10 +896,8 @@ static enum ucode_state request_microcode_amd(int cpu, struct device *device,
}
ret = UCODE_ERROR;
if (*(u32 *)fw->data != UCODE_MAGIC) {
pr_err("invalid magic value (0x%08x)\n", *(u32 *)fw->data);
if (!verify_container(fw->data, fw->size, false))
goto fw_release;
}
ret = load_microcode_amd(bsp, c->x86, fw->data, fw->size);