Signed-off-by: Ben Pfaff <b...@nicira.com>
---
Arguably the call to dp_upcall() in dp_process_received_packet()
should also be followed by consume_skb() instead of kfree_skb().
And I see that fragment() in vport-capwap.c uses kfree_skb() from
its "out" label that is used for both success and failure paths.
diff --git a/datapath/datapath.c b/datapath/datapath.c
index cd29482..01d0dd7 100644
--- a/datapath/datapath.c
+++ b/datapath/datapath.c
@@ -389,7 +389,7 @@ int dp_upcall(struct datapath *dp, struct sk_buff *skb,
/* Free GSO-segments */
do {
next = segs->next;
- kfree_skb(segs);
+ consume_skb(segs);
} while ((segs = next) != NULL);
}
--
1.7.2.5
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
