[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 30/32] target-i386: fix PSE36 mode
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PULL 30/32] target-i386: fix PSE36 mode |
Date: |
Tue, 9 Feb 2016 13:13:49 +0100 |
(pde & 0x1fe000) is a 32-bit integer; when shifting it
into bits 39-32 the result is zero. Fix it by making the
mask (and thus the result of the AND) a 64-bit integer.
Reported by Coverity.
Signed-off-by: Paolo Bonzini <address@hidden>
---
target-i386/helper.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/target-i386/helper.c b/target-i386/helper.c
index 81568c8..3802ed9 100644
--- a/target-i386/helper.c
+++ b/target-i386/helper.c
@@ -861,7 +861,7 @@ int x86_cpu_handle_mmu_fault(CPUState *cs, vaddr addr,
/* Bits 20-13 provide bits 39-32 of the address, bit 21 is
reserved.
* Leave bits 20-13 in place for setting accessed/dirty bits below.
*/
- pte = pde | ((pde & 0x1fe000) << (32 - 13));
+ pte = pde | ((pde & 0x1fe000LL) << (32 - 13));
rsvd_mask = 0x200000;
goto do_check_protect_pse36;
}
@@ -1056,7 +1056,7 @@ hwaddr x86_cpu_get_phys_page_debug(CPUState *cs, vaddr
addr)
if (!(pde & PG_PRESENT_MASK))
return -1;
if ((pde & PG_PSE_MASK) && (env->cr[4] & CR4_PSE_MASK)) {
- pte = pde | ((pde & 0x1fe000) << (32 - 13));
+ pte = pde | ((pde & 0x1fe000LL) << (32 - 13));
page_size = 4096 * 1024;
} else {
/* page directory entry */
--
2.5.0
[Qemu-devel] [PULL 32/32] qemu-char, io: fix ordering of arguments for UDP socket creation, Paolo Bonzini, 2016/02/09
[Qemu-devel] [PULL 08/32] hw: Add support for LSI SAS1068 (mptsas) device, Paolo Bonzini, 2016/02/09
Re: [Qemu-devel] [PULL v2 00/32] Misc changes for 2016-02-08, Peter Maydell, 2016/02/09