Re: [PATCH net 1/2] openvswitch: Fix helper reference leak

2015-12-23 Thread Joe Stringer
On 9 December 2015 at 15:33, Pravin Shelar wrote: > On Wed, Dec 9, 2015 at 3:10 PM, Joe Stringer wrote: >> On 9 December 2015 at 14:50, Pravin Shelar wrote: >>> On Wed, Dec 9, 2015 at 2:07 PM, Joe Stringer wrote: If the

Re: [PATCH net 1/2] openvswitch: Fix helper reference leak

2015-12-11 Thread David Miller
From: Joe Stringer Date: Wed, 9 Dec 2015 14:07:39 -0800 > If the actions (re)allocation fails, or the actions list is larger than the > maximum size, and the conntrack action is the last action when these > problems are hit, then references to helper modules may be leaked. Fix >

[PATCH net 1/2] openvswitch: Fix helper reference leak

2015-12-09 Thread Joe Stringer
If the actions (re)allocation fails, or the actions list is larger than the maximum size, and the conntrack action is the last action when these problems are hit, then references to helper modules may be leaked. Fix the issue. Fixes: cae3a2627520 ("openvswitch: Allow attaching helpers to ct

Re: [PATCH net 1/2] openvswitch: Fix helper reference leak

2015-12-09 Thread Pravin Shelar
On Wed, Dec 9, 2015 at 2:07 PM, Joe Stringer wrote: > If the actions (re)allocation fails, or the actions list is larger than the > maximum size, and the conntrack action is the last action when these > problems are hit, then references to helper modules may be leaked. Fix > the

Re: [PATCH net 1/2] openvswitch: Fix helper reference leak

2015-12-09 Thread Pravin Shelar
On Wed, Dec 9, 2015 at 3:10 PM, Joe Stringer wrote: > On 9 December 2015 at 14:50, Pravin Shelar wrote: >> On Wed, Dec 9, 2015 at 2:07 PM, Joe Stringer wrote: >>> If the actions (re)allocation fails, or the actions list is larger than the >>>

Re: [PATCH net 1/2] openvswitch: Fix helper reference leak

2015-12-09 Thread Joe Stringer
On 9 December 2015 at 14:50, Pravin Shelar wrote: > On Wed, Dec 9, 2015 at 2:07 PM, Joe Stringer wrote: >> If the actions (re)allocation fails, or the actions list is larger than the >> maximum size, and the conntrack action is the last action when these >>