Commit 91d6cd12 (ofproto-dpif: Move send_packet() to ofproto-dpif-xlate
module.) introduced the bug that uninit the "struct xlate_out" when
the struct is not initialized. This commit fixes the bug.
Reported-by: Simon Horman <ho...@verge.net.au>
Signed-off-by: Alex Wang <al...@nicira.com>
---
ofproto/ofproto-dpif-xlate.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
index 4fb0d5e..964fb15 100644
--- a/ofproto/ofproto-dpif-xlate.c
+++ b/ofproto/ofproto-dpif-xlate.c
@@ -2899,8 +2899,7 @@ xlate_send_packet(const struct ofport_dpif *ofport,
struct ofpbuf *packet)
xout.odp_actions.data, xout.odp_actions.size,
packet, (xout.slow & SLOW_ACTION) != 0);
ovs_rwlock_unlock(&xlate_rwlock);
-
-out:
xlate_out_uninit(&xout);
+out:
return error;
}
--
1.7.9.5
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
