On Thu, Dec 11, 2014 at 01:19:27PM +0530, Amit Shah wrote: > This was reported for this warning: > > hw/virtio/virtio-rng.c:150:31: warning: logical not is only applied to > the left hand side of comparison [-Wlogical-not-parentheses] > > Reported-by: dcb > Suggested-by: dcb > Bug: https://bugs.launchpad.net/qemu/+bug/1393486 > Signed-off-by: Amit Shah <amit.s...@redhat.com> > --- > hw/virtio/virtio-rng.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/virtio/virtio-rng.c b/hw/virtio/virtio-rng.c > index 473c044..06e7178 100644 > --- a/hw/virtio/virtio-rng.c > +++ b/hw/virtio/virtio-rng.c > @@ -149,7 +149,7 @@ static void virtio_rng_device_realize(DeviceState *dev, > Error **errp) > VirtIORNG *vrng = VIRTIO_RNG(dev); > Error *local_err = NULL; > > - if (!vrng->conf.period_ms > 0) { > + if (vrng->conf.period_ms <= 0) {
Thanks.
Reviewed-by: Amos Kong <ak...@redhat.com>
> error_setg(errp, "'period' parameter expects a positive integer");
> return;
> }
> --
> 2.1.0
--
Amos.
signature.asc
Description: Digital signature
