In case NDEBUG is defined, assert() expands to nothing and vhost_net_set_vnet_endian() doesn't get called...
Suggested-by: Thomas Huth <[email protected]> Signed-off-by: Greg Kurz <[email protected]> --- hw/net/vhost_net.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Michael, Yet another fix for the cross-endian vhost series... please apply. -- Greg 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)
