diff options
Diffstat (limited to 'results/classifier/108/other/1393440')
| -rw-r--r-- | results/classifier/108/other/1393440 | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/results/classifier/108/other/1393440 b/results/classifier/108/other/1393440 new file mode 100644 index 000000000..dd7688c70 --- /dev/null +++ b/results/classifier/108/other/1393440 @@ -0,0 +1,36 @@ +device: 0.853 +network: 0.773 +socket: 0.749 +files: 0.681 +performance: 0.658 +PID: 0.614 +vnc: 0.607 +semantic: 0.591 +permissions: 0.561 +graphic: 0.556 +debug: 0.483 +KVM: 0.336 +boot: 0.275 +other: 0.187 + +pcie.c:148: possible error in OR expression ? + +[qemu/hw/pci/pcie.c:148] -> [qemu/hw/pci/pcie.c:148]: (style) Same expression on both sides of '|'. + + pci_long_test_and_set_mask(dev->w1cmask + pos + PCI_EXP_DEVSTA, + PCI_EXP_DEVSTA_CED | PCI_EXP_DEVSTA_NFED | + PCI_EXP_DEVSTA_URD | PCI_EXP_DEVSTA_URD); + +I am guessing that something like + + pci_long_test_and_set_mask(dev->w1cmask + pos + PCI_EXP_DEVSTA, + PCI_EXP_DEVSTA_CED | PCI_EXP_DEVSTA_NFED | + PCI_EXP_DEVSTA_FED | PCI_EXP_DEVSTA_URD); + +was intended. + +I used static analyser cppcheck to find this bug. + +Fixed here: +http://git.qemu.org/?p=qemu.git;a=commitdiff;h=8e815eeefe205155f5 + |