blob: 178318f05bfa77be7623e8dac9ef903e87439929 (
plain) (
blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
|
device: 0.845
vnc: 0.808
ppc: 0.788
network: 0.688
register: 0.649
files: 0.623
socket: 0.581
PID: 0.511
arm: 0.468
architecture: 0.446
graphic: 0.441
semantic: 0.441
kernel: 0.431
boot: 0.385
virtual: 0.364
VMM: 0.341
TCG: 0.335
risc-v: 0.300
i386: 0.287
permissions: 0.271
x86: 0.267
debug: 0.266
KVM: 0.220
performance: 0.207
hypervisor: 0.181
mistranslation: 0.180
peripherals: 0.176
user-level: 0.115
assembly: 0.076
hw/remote/mpqemu-link.c:221: bad error checking ?
hw/remote/mpqemu-link.c:221:36: warning: logical ‘and’ of mutually exclusive tests is always false [-Wlogical-op]
Source code is
if (msg->cmd >= MPQEMU_CMD_MAX && msg->cmd < 0) {
return false;
}
Maybe better code:
if (msg->cmd >= MPQEMU_CMD_MAX || msg->cmd < 0) {
return false;
}
It might be useful to switch on gcc compiler flag -Wlogical-op
to see these warnings.
Thanks, I've reported it on the mailing list, and a patch has now been posted here:
https://lists.gnu.org/archive/html/qemu-devel/2021-05/msg02106.html
Fix has been merged now:
https://gitlab.com/qemu-project/qemu/-/commit/dcf20655ffca2b0219d2914d
|