Re: [ovs-dev] [PATCH] datapath: Fix recirc bug where skb is double freed.

2014-08-26 Thread Andy Zhou
On Tue, Aug 26, 2014 at 12:39 PM, Pravin Shelar wrote: > On Tue, Aug 26, 2014 at 11:45 AM, Andy Zhou wrote: >> On Mon, Aug 25, 2014 at 6:32 PM, Pravin Shelar wrote: >>> On Mon, Aug 25, 2014 at 3:24 PM, Andy Zhou wrote: If recirc action is the last action of a action list, the SKB triggers

Re: [ovs-dev] [PATCH] datapath: Fix recirc bug where skb is double freed.

2014-08-26 Thread Pravin Shelar
On Tue, Aug 26, 2014 at 11:45 AM, Andy Zhou wrote: > On Mon, Aug 25, 2014 at 6:32 PM, Pravin Shelar wrote: >> On Mon, Aug 25, 2014 at 3:24 PM, Andy Zhou wrote: >>> If recirc action is the last action of a action list, the SKB triggers >>> the recirc will be freed twice. This patch fixes this bug

Re: [ovs-dev] [PATCH] datapath: Fix recirc bug where skb is double freed.

2014-08-26 Thread Andy Zhou
On Mon, Aug 25, 2014 at 6:32 PM, Pravin Shelar wrote: > On Mon, Aug 25, 2014 at 3:24 PM, Andy Zhou wrote: >> If recirc action is the last action of a action list, the SKB triggers >> the recirc will be freed twice. This patch fixes this bug. >> >> Reported-by: Justin Pettit >> Signed-off-by: And

Re: [ovs-dev] [PATCH] datapath: Fix recirc bug where skb is double freed.

2014-08-25 Thread Pravin Shelar
On Mon, Aug 25, 2014 at 3:24 PM, Andy Zhou wrote: > If recirc action is the last action of a action list, the SKB triggers > the recirc will be freed twice. This patch fixes this bug. > > Reported-by: Justin Pettit > Signed-off-by: Andy Zhou > --- > datapath/actions.c | 6 ++ > 1 file chang

[ovs-dev] [PATCH] datapath: Fix recirc bug where skb is double freed.

2014-08-25 Thread Andy Zhou
If recirc action is the last action of a action list, the SKB triggers the recirc will be freed twice. This patch fixes this bug. Reported-by: Justin Pettit Signed-off-by: Andy Zhou --- datapath/actions.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/datapath/actions.c b/datapath/ac