diff options
| -rw-r--r-- | include/net/netfilter/nf_conntrack.h | 9 | ||||
| -rw-r--r-- | net/netfilter/nf_conntrack_extend.c | 3 |
2 files changed, 4 insertions, 8 deletions
diff --git a/include/net/netfilter/nf_conntrack.h b/include/net/netfilter/nf_conntrack.h index bb9fc852e973..4a0496aa32d5 100644 --- a/include/net/netfilter/nf_conntrack.h +++ b/include/net/netfilter/nf_conntrack.h | |||
| @@ -65,14 +65,7 @@ union nf_conntrack_help { | |||
| 65 | #include <linux/timer.h> | 65 | #include <linux/timer.h> |
| 66 | 66 | ||
| 67 | #ifdef CONFIG_NETFILTER_DEBUG | 67 | #ifdef CONFIG_NETFILTER_DEBUG |
| 68 | #define NF_CT_ASSERT(x) \ | 68 | #define NF_CT_ASSERT(x) WARN_ON(!(x)) |
| 69 | do { \ | ||
| 70 | if (!(x)) \ | ||
| 71 | /* Wooah! I'm tripping my conntrack in a frenzy of \ | ||
| 72 | netplay... */ \ | ||
| 73 | printk("NF_CT_ASSERT: %s:%i(%s)\n", \ | ||
| 74 | __FILE__, __LINE__, __FUNCTION__); \ | ||
| 75 | } while(0) | ||
| 76 | #else | 69 | #else |
| 77 | #define NF_CT_ASSERT(x) | 70 | #define NF_CT_ASSERT(x) |
| 78 | #endif | 71 | #endif |
diff --git a/net/netfilter/nf_conntrack_extend.c b/net/netfilter/nf_conntrack_extend.c index 2bd9963b5b3e..bcc19fa4ed1e 100644 --- a/net/netfilter/nf_conntrack_extend.c +++ b/net/netfilter/nf_conntrack_extend.c | |||
| @@ -71,6 +71,9 @@ void *__nf_ct_ext_add(struct nf_conn *ct, enum nf_ct_ext_id id, gfp_t gfp) | |||
| 71 | int i, newlen, newoff; | 71 | int i, newlen, newoff; |
| 72 | struct nf_ct_ext_type *t; | 72 | struct nf_ct_ext_type *t; |
| 73 | 73 | ||
| 74 | /* Conntrack must not be confirmed to avoid races on reallocation. */ | ||
| 75 | NF_CT_ASSERT(!nf_ct_is_confirmed(ct)); | ||
| 76 | |||
| 74 | if (!ct->ext) | 77 | if (!ct->ext) |
| 75 | return nf_ct_ext_create(&ct->ext, id, gfp); | 78 | return nf_ct_ext_create(&ct->ext, id, gfp); |
| 76 | 79 | ||
