All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH NF_CONNTRACK 0/8]: conflict symbols and getorigdst issue/bugs
@ 2005-03-22 13:33 Yasuyuki KOZAKAI
  0 siblings, 0 replies; 2+ messages in thread
From: Yasuyuki KOZAKAI @ 2005-03-22 13:33 UTC (permalink / raw)
  To: netfilter-devel

Hi, Patrick,

These are patches for nf_conntrack which
	- unify common enums/flags in ip_conntrack and nf_conntrack,
	  as we talked,
	- fix other conflicts by changing he name of them in nf_conntrack,
	- change SO_ORIGINAL_DST to IPV6_ORIGINAL_DST in netfilter_ipv6.h
	  SO_* are socket level option names, not IP level. This doesn't
	  change netfilter_ipv4.h because of compatibility issue.
	- fix the bug which cannot find the target conntrack.

BTW, getorigdst() for IPv6 connection is really useful ? IPv6 connections
are not NATed, then I think it's meaningless to call it.
If it looks like it's useless, deleting it is fine with me.
If not, please apply patches I send (08, 09).

Moreover, IPV6_ORIGINAL_DST is 80 in the current. Is this value really
appropriate ?

-----------------------------------------------------------------
Yasuyuki KOZAKAI @ USAGI Project <yasuyuki.kozakai@toshiba.co.jp>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH NF_CONNTRACK 0/8]: conflict symbols and getorigdst issue/bugs
       [not found] <200503221323.j2MDNfXS028596@toshiba.co.jp>
@ 2005-04-17 21:49 ` Patrick McHardy
  0 siblings, 0 replies; 2+ messages in thread
From: Patrick McHardy @ 2005-04-17 21:49 UTC (permalink / raw)
  To: Yasuyuki KOZAKAI; +Cc: Netfilter Development Mailinglist

Hi Yasuyuki,

Yasuyuki KOZAKAI wrote:

> These are patches for nf_conntrack which
> 	- unify common enums/flags in ip_conntrack and nf_conntrack,
> 	  as we talked,
> 	- fix other conflicts by changing he name of them in nf_conntrack,
> 	- change SO_ORIGINAL_DST to IPV6_ORIGINAL_DST in netfilter_ipv6.h
> 	  SO_* are socket level option names, not IP level. This doesn't
> 	  change netfilter_ipv4.h because of compatibility issue.
> 	- fix the bug which cannot find the target conntrack.
> 
> BTW, getorigdst() for IPv6 connection is really useful ? IPv6 connections
> are not NATed, then I think it's meaningless to call it.
> If it looks like it's useless, deleting it is fine with me.
> If not, please apply patches I send (08, 09).

sorry for takeing so long with this. I've applied patches 01-07 and
removed getorigdst(), thanks a lot.

Regards
Patrick

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-04-17 21:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <200503221323.j2MDNfXS028596@toshiba.co.jp>
2005-04-17 21:49 ` [PATCH NF_CONNTRACK 0/8]: conflict symbols and getorigdst issue/bugs Patrick McHardy
2005-03-22 13:33 Yasuyuki KOZAKAI

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.