diff options
author | Eric Paris <eparis@redhat.com> | 2010-10-19 18:17:32 -0400 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2010-10-20 19:13:00 -0400 |
commit | ff660c80d00b52287f1f67ee6c115dc0057bcdde (patch) | |
tree | 63a0e4c69b9d78b5b81156029d5599dc2f3bf0a4 | |
parent | 845ca30fe9691f1bab7cfbf30b6d11c944eb4abd (diff) |
secmark: fix config problem when CONFIG_NF_CONNTRACK_SECMARK is not set
When CONFIG_NF_CONNTRACK_SECMARK is not set we accidentally attempt to use
the secmark fielf of struct nf_conn. Problem is when that config isn't set
the field doesn't exist. whoops. Wrap the incorrect usage in the config.
Signed-off-by: Eric Paris <eparis@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
-rw-r--r-- | net/netfilter/nf_conntrack_netlink.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/netfilter/nf_conntrack_netlink.c b/net/netfilter/nf_conntrack_netlink.c index b3c628555cf3..146476c6441a 100644 --- a/net/netfilter/nf_conntrack_netlink.c +++ b/net/netfilter/nf_conntrack_netlink.c | |||
@@ -582,9 +582,11 @@ ctnetlink_conntrack_event(unsigned int events, struct nf_ct_event *item) | |||
582 | && ctnetlink_dump_helpinfo(skb, ct) < 0) | 582 | && ctnetlink_dump_helpinfo(skb, ct) < 0) |
583 | goto nla_put_failure; | 583 | goto nla_put_failure; |
584 | 584 | ||
585 | #ifdef CONFIG_NF_CONNTRACK_SECMARK | ||
585 | if ((events & (1 << IPCT_SECMARK) || ct->secmark) | 586 | if ((events & (1 << IPCT_SECMARK) || ct->secmark) |
586 | && ctnetlink_dump_secctx(skb, ct) < 0) | 587 | && ctnetlink_dump_secctx(skb, ct) < 0) |
587 | goto nla_put_failure; | 588 | goto nla_put_failure; |
589 | #endif | ||
588 | 590 | ||
589 | if (events & (1 << IPCT_RELATED) && | 591 | if (events & (1 << IPCT_RELATED) && |
590 | ctnetlink_dump_master(skb, ct) < 0) | 592 | ctnetlink_dump_master(skb, ct) < 0) |