netdev
[Top] [All Lists]

Re: [PATCH] IPv6: Functions Clean-up

To: YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@xxxxxxxxxxxxxx>
Subject: Re: [PATCH] IPv6: Functions Clean-up
From: Andras Kis-Szabo <kisza@xxxxxxxxxxxxxxxx>
Date: Sun, 03 Nov 2002 15:27:42 +0100
Cc: linux-kernel@xxxxxxxxxxxxxxx, Netdev <netdev@xxxxxxxxxxx>, Netfilter Devel <netfilter-devel@xxxxxxxxxxxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxx>, kuznet@xxxxxxxxxxxxx, usagi@xxxxxxxxxxxxxx
In-reply-to: <20021103.221658.52523847.yoshfuji@xxxxxxxxxxxxxx>
References: <20021103.115427.104445233.yoshfuji@xxxxxxxxxxxxxx> <1036328414.1048.3.camel@arwen> <20021103.221658.52523847.yoshfuji@xxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
Hi,

> /*
>  * Taken from net/ipv6/ip6_output.c
>  * We should use the one there, but is defined static
>  * so we put this just here and let the things as
>  * they are now.
>  *
>  * If that one is modified, this one should be modified too.
>  */
> 
> So, the reason why the copy of route6_me_harder() 
> lives there is because net/ipv6/ip6_output.c has not been 
> exported it.
Ok, ok! I have not got problems with this any more! :)
It is even worse that the route6_me_harder() depends on the
CONFIG_NETFILTER option, so this function comes from the Netfilter. In
this case it is an unwanted code-duplication :(
 
> Is this something to do with parser of extension headers?
> Parser of extension headers is different thing, isn't it?
Yes, it is a different thing with different code.
 
Regards,

kisza

-- 
    Andras Kis-Szabo       Security Development, Design and Audit
-------------------------/        Zorp, NetFilter and IPv6
 kisza@xxxxxxxxxxxxxxxx /------------------------------------------->


<Prev in Thread] Current Thread [Next in Thread>