diff options
| -rw-r--r-- | include/net/netfilter/nf_conntrack_extend.h | 1 | ||||
| -rw-r--r-- | net/ipv4/netfilter/nf_nat_core.c | 3 | ||||
| -rw-r--r-- | net/netfilter/nf_conntrack_extend.c | 9 |
3 files changed, 10 insertions, 3 deletions
diff --git a/include/net/netfilter/nf_conntrack_extend.h b/include/net/netfilter/nf_conntrack_extend.h index f736e842977f..f80c0ed6d870 100644 --- a/include/net/netfilter/nf_conntrack_extend.h +++ b/include/net/netfilter/nf_conntrack_extend.h | |||
| @@ -15,6 +15,7 @@ enum nf_ct_ext_id | |||
| 15 | 15 | ||
| 16 | /* Extensions: optional stuff which isn't permanently in struct. */ | 16 | /* Extensions: optional stuff which isn't permanently in struct. */ |
| 17 | struct nf_ct_ext { | 17 | struct nf_ct_ext { |
| 18 | struct rcu_head rcu; | ||
| 18 | u8 offset[NF_CT_EXT_NUM]; | 19 | u8 offset[NF_CT_EXT_NUM]; |
| 19 | u8 len; | 20 | u8 len; |
| 20 | char data[0]; | 21 | char data[0]; |
diff --git a/net/ipv4/netfilter/nf_nat_core.c b/net/ipv4/netfilter/nf_nat_core.c index 04578593e100..d2a887fc8d9b 100644 --- a/net/ipv4/netfilter/nf_nat_core.c +++ b/net/ipv4/netfilter/nf_nat_core.c | |||
| @@ -556,7 +556,6 @@ static void nf_nat_cleanup_conntrack(struct nf_conn *ct) | |||
| 556 | 556 | ||
| 557 | spin_lock_bh(&nf_nat_lock); | 557 | spin_lock_bh(&nf_nat_lock); |
| 558 | hlist_del_rcu(&nat->bysource); | 558 | hlist_del_rcu(&nat->bysource); |
| 559 | nat->ct = NULL; | ||
| 560 | spin_unlock_bh(&nf_nat_lock); | 559 | spin_unlock_bh(&nf_nat_lock); |
| 561 | } | 560 | } |
| 562 | 561 | ||
| @@ -570,8 +569,8 @@ static void nf_nat_move_storage(void *new, void *old) | |||
| 570 | return; | 569 | return; |
| 571 | 570 | ||
| 572 | spin_lock_bh(&nf_nat_lock); | 571 | spin_lock_bh(&nf_nat_lock); |
| 573 | hlist_replace_rcu(&old_nat->bysource, &new_nat->bysource); | ||
| 574 | new_nat->ct = ct; | 572 | new_nat->ct = ct; |
| 573 | hlist_replace_rcu(&old_nat->bysource, &new_nat->bysource); | ||
| 575 | spin_unlock_bh(&nf_nat_lock); | 574 | spin_unlock_bh(&nf_nat_lock); |
| 576 | } | 575 | } |
| 577 | 576 | ||
diff --git a/net/netfilter/nf_conntrack_extend.c b/net/netfilter/nf_conntrack_extend.c index bcc19fa4ed1e..8a3f8b34e466 100644 --- a/net/netfilter/nf_conntrack_extend.c +++ b/net/netfilter/nf_conntrack_extend.c | |||
| @@ -59,12 +59,19 @@ nf_ct_ext_create(struct nf_ct_ext **ext, enum nf_ct_ext_id id, gfp_t gfp) | |||
| 59 | if (!*ext) | 59 | if (!*ext) |
| 60 | return NULL; | 60 | return NULL; |
| 61 | 61 | ||
| 62 | INIT_RCU_HEAD(&(*ext)->rcu); | ||
| 62 | (*ext)->offset[id] = off; | 63 | (*ext)->offset[id] = off; |
| 63 | (*ext)->len = len; | 64 | (*ext)->len = len; |
| 64 | 65 | ||
| 65 | return (void *)(*ext) + off; | 66 | return (void *)(*ext) + off; |
| 66 | } | 67 | } |
| 67 | 68 | ||
| 69 | static void __nf_ct_ext_free_rcu(struct rcu_head *head) | ||
| 70 | { | ||
| 71 | struct nf_ct_ext *ext = container_of(head, struct nf_ct_ext, rcu); | ||
| 72 | kfree(ext); | ||
| 73 | } | ||
| 74 | |||
| 68 | void *__nf_ct_ext_add(struct nf_conn *ct, enum nf_ct_ext_id id, gfp_t gfp) | 75 | void *__nf_ct_ext_add(struct nf_conn *ct, enum nf_ct_ext_id id, gfp_t gfp) |
| 69 | { | 76 | { |
| 70 | struct nf_ct_ext *new; | 77 | struct nf_ct_ext *new; |
| @@ -106,7 +113,7 @@ void *__nf_ct_ext_add(struct nf_conn *ct, enum nf_ct_ext_id id, gfp_t gfp) | |||
| 106 | (void *)ct->ext + ct->ext->offset[i]); | 113 | (void *)ct->ext + ct->ext->offset[i]); |
| 107 | rcu_read_unlock(); | 114 | rcu_read_unlock(); |
| 108 | } | 115 | } |
| 109 | kfree(ct->ext); | 116 | call_rcu(&ct->ext->rcu, __nf_ct_ext_free_rcu); |
| 110 | ct->ext = new; | 117 | ct->ext = new; |
| 111 | } | 118 | } |
| 112 | 119 | ||
