linux-kernel - Re: [netfilter-core] [2.6 patch] net/*/nf_conntrack

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4570BBFE.8090209@trash.net>
Date:	Sat, 02 Dec 2006 00:34:22 +0100
From:	Patrick McHardy <kaber@...sh.net>
To:	Adrian Bunk <bunk@...sta.de>
CC:	coreteam@...filter.org, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [netfilter-core] [2.6 patch] net/*/nf_conntrack_*.c:possible
 cleanups

Adrian Bunk wrote:
> This patch contains the following possible cleanups:
> - make the following needlessly global functions static:
>   - net/netfilter/nf_conntrack_core.c: nf_conntrack_register_cache()
>   - net/netfilter/nf_conntrack_core.c: nf_conntrack_unregister_cache()
>   - net/netfilter/nf_conntrack_core.c: __nf_conntrack_attach()
>   - net/netfilter/nf_conntrack_core.c: set_hashsize()
>   - net/netfilter/nf_conntrack_proto_sctp.c: nf_conntrack_proto_sctp_init()
>   - net/netfilter/nf_conntrack_proto_sctp.c: nf_conntrack_proto_sctp_fini()
> - make the following needlessly global variables/locks/structs static:
>   - net/ipv6/netfilter/nf_conntrack_reasm.c: nf_ct_frag6_secret_interval
>   - net/ipv6/netfilter/nf_conntrack_reasm.c: nf_ct_frag6_mem
>   - net/netfilter/nf_conntrack_core.c: nf_ct_cache_lock
>   - net/netfilter/nf_conntrack_proto_sctp.c: nf_conntrack_protocol_sctp4
>   - net/netfilter/nf_conntrack_proto_sctp.c: nf_conntrack_protocol_sctp6
> - #if 0 the following unused global functions:
>   - net/ipv6/netfilter/nf_conntrack_reasm.c: nf_ct_frag6_kfree_frags()
>   - net/netfilter/nf_conntrack_core.c: nf_conntrack_tuple_taken()
>   - net/netfilter/nf_conntrack_core.c: nf_ct_invert_tuplepr()
> - remove the following unused or write-only variables:
>   - net/ipv4/netfilter/nf_conntrack_l3proto_ipv4.c: nat_module_is_loaded
>   - net/ipv6/netfilter/nf_conntrack_reasm.c: nf_ct_frag6_nqueues
> - remove the following unused hooks:
>   - net/netfilter/nf_conntrack_core.c: nf_conntrack_destroyed()
>   - net/netfilter/nf_conntrack_ftp.c: nf_nat_ftp_hook()
> - remove the following unused EXPORT_SYMBOL's:
>   - nf_ct_iterate_cleanup
>   - nf_ct_protos
>   - nf_ct_l3protos
> 
> Please review which of these changes make sense and which might conflict 
> with pending patches.

Thanks Adrian. We have a large nf_conntrack merge coming up, which
conflicts with this patch and includes new users of some of these
symbols. Please send your patch again once these changes have
been merged.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/