Hi, On Wednesday 10 January 2007 07:47, Patrick McHardy wrote: > > diff --git a/net/ipv4/route.c b/net/ipv4/route.c > > index 537b976..bb1158a 100644 > > --- a/net/ipv4/route.c > > +++ b/net/ipv4/route.c > > @@ -2498,11 +2498,6 @@ #endif > > ZERONET(oldflp->fl4_src)) > > goto out; > > > > - /* It is equivalent to inet_addr_type(saddr) == RTN_LOCAL */ > > - dev_out = ip_dev_find(oldflp->fl4_src); > > - if (dev_out == NULL) > > - goto out; > > - > > I'm not sure how exactly this is used by applications, but couldn't you > restrict this to sockets without freebind?
I'll try to do so in the next incarnation of the patches. Thanks for the comment, it'd ineed be safer to do so. BTW, could anyone shed some light on exactly why that check is necessary? As far as I can see it prevents packets with a non-local source address being routed -- but I fail to see why we need to prevent that. -- Regards, Krisztian Kovacs - 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
