[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2] vhost_net: fix misuse of assert()
From: |
Greg Kurz |
Subject: |
[Qemu-devel] [PATCH v2] vhost_net: fix misuse of assert() |
Date: |
Fri, 19 Jun 2015 14:03:43 +0200 |
User-agent: |
StGit/0.17.1-dirty |
>From assert(3): "assert() is implemented as a macro; if the expression tested
has side-effects, program behavior will be different depending on whether
NDEBUG is defined."
Even if QEMU isn't compiled with NDEBUG, it is bad practice to put bits
with a relevant functionnal meaning in assert().
This is a readability enhancement. The patch does not fix any bug.
Suggested-by: Thomas Huth <address@hidden>
Signed-off-by: Greg Kurz <address@hidden>
---
hw/net/vhost_net.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
v2: rewrote commit log
diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
index f505c91d420a..6d3b7879f8ef 100644
--- a/hw/net/vhost_net.c
+++ b/hw/net/vhost_net.c
@@ -376,7 +376,8 @@ void vhost_net_stop(VirtIODevice *dev, NetClientState *ncs,
}
assert(r >= 0);
- assert(vhost_net_set_vnet_endian(dev, ncs[0].peer, false) >= 0);
+ r = vhost_net_set_vnet_endian(dev, ncs[0].peer, false);
+ assert(r >= 0);
}
void vhost_net_cleanup(struct vhost_net *net)
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-devel] [PATCH v2] vhost_net: fix misuse of assert(),
Greg Kurz <=