On 24 October 2015 at 01:20, Florian Westphal wrote:
> Joe Stringer wrote:
>> err:
>> + if (err)
>> + kfree_skb(skb);
>> skb_push(skb, nh_ofs);
>
> That looks... wrong :-}
D'oh. Teaches me for last minute adjustments. I'll resend.
--
To unsubscribe from this list: send the
Joe Stringer wrote:
> err:
> + if (err)
> + kfree_skb(skb);
> skb_push(skb, nh_ofs);
That looks... wrong :-}
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel
If ip_defrag() returns an error other than -EINPROGRESS, then the skb is
freed. When handle_fragments() passes this back up to
do_execute_actions(), it will be freed again. Prevent this double free
by never freeing the skb in do_execute_actions() for errors returned by
ovs_ct_execute. Always free i