summary refs log tree commit diff stats
path: root/results/scraper/launchpad/1630527
blob: c3538ec3afdf0c711e0fb1125abc513b52a9d603 (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
qemu/hw/i386/amd_iommu.c:188: possible bad shift ?

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;

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


Released with v2.8