Re: [PATCHv2 3/7] openvswitch: Fix skb leak in ovs_fragment()

2015-10-01 Thread Pravin Shelar
On Thu, Oct 1, 2015 at 1:53 PM, Joe Stringer wrote: > If ovs_fragment() was unable to fragment the skb due to an L2 header > that exceeds the supported length, skbs would be leaked. Fix the bug. > > Fixes: 7f8a436 "openvswitch: Add conntrack action" > Signed-off-by: Joe Stringer > --- > v2: Drop

[PATCHv2 3/7] openvswitch: Fix skb leak in ovs_fragment()

2015-10-01 Thread Joe Stringer
If ovs_fragment() was unable to fragment the skb due to an L2 header that exceeds the supported length, skbs would be leaked. Fix the bug. Fixes: 7f8a436 "openvswitch: Add conntrack action" Signed-off-by: Joe Stringer --- v2: Drop if condition, return in success case. ---

[PATCHv2 3/7] openvswitch: Fix skb leak in ovs_fragment()

2015-10-01 Thread Joe Stringer
If ovs_fragment() was unable to fragment the skb due to an L2 header that exceeds the supported length, skbs would be leaked. Fix the bug. Fixes: 7f8a436 "openvswitch: Add conntrack action" Signed-off-by: Joe Stringer --- v2: Drop if condition, return in success case. ---

Re: [PATCHv2 3/7] openvswitch: Fix skb leak in ovs_fragment()

2015-10-01 Thread Pravin Shelar
On Thu, Oct 1, 2015 at 1:53 PM, Joe Stringer wrote: > If ovs_fragment() was unable to fragment the skb due to an L2 header > that exceeds the supported length, skbs would be leaked. Fix the bug. > > Fixes: 7f8a436 "openvswitch: Add conntrack action" > Signed-off-by: Joe