Messages in this thread | | | Date | Fri, 13 Apr 2012 15:22:50 +0900 | From | Simon Horman <> | Subject | Re: [PATCH] netfilter: ipvs: Verify that IP_VS protocol has been registered |
| |
On Fri, Apr 13, 2012 at 01:11:50PM +0900, Simon Horman wrote: > On Fri, Apr 13, 2012 at 02:54:13AM +0200, Sasha Levin wrote: > > On Thu, Apr 12, 2012 at 1:46 AM, Pablo Neira Ayuso <pablo@netfilter.org> wrote: > > >> If you return here, I think you'll leave things in inconsistent state, > > >> ie. the tcp protocol is registered. You have to unregister it before > > >> leaving. > > > > I thought that the cleanup callback is getting called for failed init > > calls, if that's not the case then we can probably call it ourselves > > if any of these failed. > > Good point. In any case, I think that I have found a new problem.
[sni@]
I have confirmed that the cleanup code is called on error, so I think that your existing patch is fine.
Acked-by: Simon Horman <horms@verge.net.au>
Pablo, please consider taking this change. The other problem I raised, though related, can be handled separately as it won't occur if the ftp helper isn't initialised.
| |