|
 |
bba56f |
From b470757774a776e2bcbff50c7ae7f40323d10064 Mon Sep 17 00:00:00 2001
|
|
|
bba56f |
Message-Id: <b470757774a776e2bcbff50c7ae7f40323d10064@dist-git>
|
|
|
bba56f |
From: Michal Privoznik <mprivozn@redhat.com>
|
|
|
bba56f |
Date: Thu, 7 Mar 2019 14:17:01 +0100
|
|
|
bba56f |
Subject: [PATCH] cpu: Don't access invalid memory in virCPUx86Translate
|
|
|
bba56f |
MIME-Version: 1.0
|
|
|
bba56f |
Content-Type: text/plain; charset=UTF-8
|
|
|
bba56f |
Content-Transfer-Encoding: 8bit
|
|
|
bba56f |
|
|
|
bba56f |
Problem is that if there are no signatures for a CPU, then we
|
|
|
bba56f |
still allocate cpu->signatures (even though with size 0). Later,
|
|
|
bba56f |
we access cpu->signatures[0] if cpu->signatures is not NULL.
|
|
|
bba56f |
|
|
|
bba56f |
Invalid read of size 4
|
|
|
bba56f |
at 0x5F439D7: virCPUx86Translate (cpu_x86.c:2930)
|
|
|
bba56f |
by 0x5F3C239: virCPUTranslate (cpu.c:927)
|
|
|
bba56f |
by 0x57CE7A1: qemuProcessUpdateGuestCPU (qemu_process.c:5870)
|
|
|
bba56f |
...
|
|
|
bba56f |
Address 0xf752d40 is 0 bytes after a block of size 0 alloc'd
|
|
|
bba56f |
at 0x4C30EC6: calloc (vg_replace_malloc.c:711)
|
|
|
bba56f |
by 0x5DBDE4E: virAllocN (viralloc.c:190)
|
|
|
bba56f |
by 0x5F3E4FA: x86ModelCopySignatures (cpu_x86.c:990)
|
|
|
bba56f |
by 0x5F3E60F: x86ModelCopy (cpu_x86.c:1008)
|
|
|
bba56f |
by 0x5F3E7CB: x86ModelFromCPU (cpu_x86.c:1068)
|
|
|
bba56f |
by 0x5F4397E: virCPUx86Translate (cpu_x86.c:2922)
|
|
|
bba56f |
by 0x5F3C239: virCPUTranslate (cpu.c:927)
|
|
|
bba56f |
by 0x57CE7A1: qemuProcessUpdateGuestCPU (qemu_process.c:5870)
|
|
|
bba56f |
...
|
|
|
bba56f |
|
|
|
bba56f |
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
|
|
|
bba56f |
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
bba56f |
(cherry picked from commit 62cb9c335c43a722e81ac0a1ed6e1111ba1d428b)
|
|
|
bba56f |
|
|
|
bba56f |
https://bugzilla.redhat.com/show_bug.cgi?id=1558558
|
|
|
bba56f |
https://bugzilla.redhat.com/show_bug.cgi?id=1687515
|
|
|
bba56f |
|
|
|
bba56f |
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
|
|
|
bba56f |
Reviewed-by: Ján Tomko <jtomko@redhat.com>
|
|
|
bba56f |
---
|
|
|
bba56f |
src/cpu/cpu_x86.c | 3 +++
|
|
|
bba56f |
1 file changed, 3 insertions(+)
|
|
|
bba56f |
|
|
|
bba56f |
diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
|
|
|
bba56f |
index f6844aaa0d..466c79031d 100644
|
|
|
bba56f |
--- a/src/cpu/cpu_x86.c
|
|
|
bba56f |
+++ b/src/cpu/cpu_x86.c
|
|
|
bba56f |
@@ -1021,6 +1021,9 @@ x86ModelCopySignatures(virCPUx86ModelPtr dst,
|
|
|
bba56f |
{
|
|
|
bba56f |
size_t i;
|
|
|
bba56f |
|
|
|
bba56f |
+ if (src->nsignatures == 0)
|
|
|
bba56f |
+ return 0;
|
|
|
bba56f |
+
|
|
|
bba56f |
if (VIR_ALLOC_N(dst->signatures, src->nsignatures) < 0)
|
|
|
bba56f |
return -1;
|
|
|
bba56f |
|
|
|
bba56f |
--
|
|
|
bba56f |
2.21.0
|
|
|
bba56f |
|