diff options
| author |   Patrick McHardy <kaber@trash.net> | 2007-02-12 11:09:19 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2007-02-12 11:09:19 -0800 |
| commit | c3a47ab3e5ad62601449e4e5401352271b777e28 (patch) | |
| tree | 3b3f6810796367efa5f76209aed12bec6c6a8b17 /net/netfilter/core.c | |
| parent | Merge branch 'for-linus' of git://git390.osdl.marist.edu/pub/scm/linux-2.6 (diff) | |
| download | linux-dev-c3a47ab3e5ad62601449e4e5401352271b777e28.tar.xz linux-dev-c3a47ab3e5ad62601449e4e5401352271b777e28.zip | |
[NETFILTER]: Properly use RCU in nf_ct_attach
Use rcu_assign_pointer/rcu_dereference for ip_ct_attach pointer instead
of self-made RCU and use rcu_read_lock to make sure the conntrack module
doesn't disappear below us while calling it, since this function can be
called from outside the netfilter hooks.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/netfilter/core.c')
| -rw-r--r-- | net/netfilter/core.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/net/netfilter/core.c b/net/netfilter/core.c index 291b8c6862f1..ad24d0c005e2 100644 --- a/net/netfilter/core.c +++ b/net/netfilter/core.c @@ -248,9 +248,12 @@ void nf_ct_attach(struct sk_buff *new, struct sk_buff *skb) { void (*attach)(struct sk_buff *, struct sk_buff *); - if (skb->nfct && (attach = ip_ct_attach) != NULL) { - mb(); /* Just to be sure: must be read before executing this */ - attach(new, skb); + if (skb->nfct) { + rcu_read_lock(); + attach = rcu_dereference(ip_ct_attach); + if (attach) + attach(new, skb); + rcu_read_unlock(); } } EXPORT_SYMBOL(nf_ct_attach); |