Hi Florian,

> -----Original Message-----
> From: Florian Westphal [mailto:[email protected]]
> Sent: Wednesday, April 5, 2017 9:17 PM
> To: [email protected]
> Cc: [email protected]; [email protected]; Gao Feng
> <[email protected]>
> Subject: Re: [PATCH nf-next 1/1] netfilter: Remove useless variable
timeouts in
> init_conntrack
> 
> [email protected] <[email protected]> wrote:
> > From: Gao Feng <[email protected]>
> >
> > All l4_proto->new callbacks don't use the param timeouts currently, so
> > remove it from param list. Then the variable timeouts isn't used as
> > right value in init_conntrack, so remove it too.
> > Signed-off-by: Gao Feng <[email protected]>
> > ---
> >  include/net/netfilter/nf_conntrack_l4proto.h |  2 +-
> > net/ipv4/netfilter/nf_conntrack_proto_icmp.c |  2 +-
> >  net/netfilter/nf_conntrack_core.c            | 13 ++-----------
> >  net/netfilter/nf_conntrack_proto_dccp.c      |  2 +-
> >  net/netfilter/nf_conntrack_proto_sctp.c      |  2 +-
> >  net/netfilter/nf_conntrack_proto_tcp.c       |  2 +-
> >  net/netfilter/nf_conntrack_proto_udp.c       |  2 +-
> >  7 files changed, 8 insertions(+), 17 deletions(-)
> 
> What about net/netfilter/nf_conntrack_proto_gre.c ?
It's my fault.
I only checked the protos in nf_conntrack_ipv4.h.

Best Regards
Feng




--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to