On Fri, Nov 02, 2007 at 01:36:07AM +0200, Julian Anastasov wrote: > > Hello, > > On Thu, 1 Nov 2007, Simon Horman wrote: > > > --- net-2.6.orig/net/ipv4/ipvs/ip_vs_sync.c 2007-11-01 18:17:55.000000000 > > +0900 > > +++ net-2.6/net/ipv4/ipvs/ip_vs_sync.c 2007-11-01 18:20:30.000000000 > > +0900 > > @@ -332,7 +332,7 @@ static void ip_vs_process_message(const > > s->daddr, s->dport, > > flags, dest); > > if (dest) > > Is that correct? Sorry, I was flooded with different versions > of this patch and I'm not sure if it is the final one. > > > - atomic_dec(&dest->refcnt); > > + ip_vs_dest_get(dest); > > if (!cp) { > > IP_VS_ERR("ip_vs_conn_new failed\n"); > > return;
The ip_vs_dest_get() call shouldn't be there. I'll double check the rest of the patch. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html