[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [Bug 1630527] Re: qemu/hw/i386/amd_iommu.c:188: possible ba
From: |
T. Huth |
Subject: |
[Qemu-devel] [Bug 1630527] Re: qemu/hw/i386/amd_iommu.c:188: possible bad shift ? |
Date: |
Wed, 12 Oct 2016 14:54:26 -0000 |
Thanks for reporting this bug! Looks like this has already been fixed by this
commit here:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=1d5b128cbeeab638f772e
** Changed in: qemu
Status: New => Fix Committed
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1630527
Title:
qemu/hw/i386/amd_iommu.c:188: possible bad shift ?
Status in QEMU:
Fix Committed
Bug description:
qemu/hw/i386/amd_iommu.c:188]: (error) Shifting 32-bit value by 64
bits is undefined behaviour
Source code is
uint64_t mask = ((1 << length) - 1) << bitpos;
Maybe better code
uint64_t mask = ((1ULL << length) - 1) << bitpos;
To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1630527/+subscriptions