handle_flow_miss() transfers ownership of its packets to
send_packet_in_miss(), but its caller deletes these packets upon
return.  Future patches will remove support for un-cloned calls
altogether.  Found by inspection.

Signed-off-by: Ethan Jackson <et...@nicira.com>
---
 ofproto/ofproto-dpif.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index df7a56a..90db2ba 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -2533,7 +2533,7 @@ handle_flow_miss(struct ofproto_dpif *ofproto, struct 
flow_miss *miss,
             LIST_FOR_EACH_SAFE (packet, next_packet, list_node,
                                 &miss->packets) {
                 list_remove(&packet->list_node);
-                send_packet_in_miss(ofproto, packet, flow, false);
+                send_packet_in_miss(ofproto, packet, flow, true);
             }
 
             return;
-- 
1.7.7.1

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to