diff options
| author |   Patrick McHardy <kaber@trash.net> | 2007-02-12 11:13:58 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2007-02-12 11:13:58 -0800 |
| commit | 6b48a7d08d1bb2e3932bce1662fe411304acc18f (patch) | |
| tree | f0405cee3ff4fb3898a2a691e25cd77c31e392d8 /net/ipv4/netfilter/ip_nat_core.c | |
| parent | [NETFILTER]: nf_conntrack: fix invalid conntrack statistics RCU assumption (diff) | |
| download | linux-dev-6b48a7d08d1bb2e3932bce1662fe411304acc18f.tar.xz linux-dev-6b48a7d08d1bb2e3932bce1662fe411304acc18f.zip | |
[NETFILTER]: ip_conntrack: properly use RCU for ip_conntrack_destroyed callback
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/netfilter/ip_nat_core.c')
| -rw-r--r-- | net/ipv4/netfilter/ip_nat_core.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/net/ipv4/netfilter/ip_nat_core.c b/net/ipv4/netfilter/ip_nat_core.c index 18daabc22069..40737fdbe9a7 100644 --- a/net/ipv4/netfilter/ip_nat_core.c +++ b/net/ipv4/netfilter/ip_nat_core.c @@ -604,8 +604,8 @@ static int __init ip_nat_init(void) } /* FIXME: Man, this is a hack. <SIGH> */ - IP_NF_ASSERT(ip_conntrack_destroyed == NULL); - ip_conntrack_destroyed = &ip_nat_cleanup_conntrack; + IP_NF_ASSERT(rcu_dereference(ip_conntrack_destroyed) == NULL); + rcu_assign_pointer(ip_conntrack_destroyed, ip_nat_cleanup_conntrack); /* Initialize fake conntrack so that NAT will skip it */ ip_conntrack_untracked.status |= IPS_NAT_DONE_MASK; @@ -623,7 +623,8 @@ static int clean_nat(struct ip_conntrack *i, void *data) static void __exit ip_nat_cleanup(void) { ip_ct_iterate_cleanup(&clean_nat, NULL); - ip_conntrack_destroyed = NULL; + rcu_assign_pointer(ip_conntrack_destroyed, NULL); + synchronize_rcu(); vfree(bysource); } |