summary refs log tree commit diff stats
path: root/results/scraper/launchpad/1393486
blob: 9a103b422ca3b2947f3480ae6fd8e0833944ee09 (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
hw/virtio/virtio-rng.c:150: bad test ?

hw/virtio/virtio-rng.c:150:31: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]

    if (!vrng->conf.period_ms > 0) {
        error_setg(errp, "'period' parameter expects a positive integer");
        return;
    }

Maybe better code

    if (vrng->conf.period_ms <= 0) {
        error_setg(errp, "'period' parameter expects a positive integer");
        return;
    }

Fixed here:
http://git.qemu.org/?p=qemu.git;a=commitdiff;h=a3a292c420d2fec3c07