From patchwork Tue Jun 3 07:03:05 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arturo Borrero X-Patchwork-Id: 355192 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id BDE1514009E for ; Tue, 3 Jun 2014 17:03:19 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753397AbaFCHDS (ORCPT ); Tue, 3 Jun 2014 03:03:18 -0400 Received: from smtp3.cica.es ([150.214.5.190]:35438 "EHLO smtp.cica.es" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932076AbaFCHDR (ORCPT ); Tue, 3 Jun 2014 03:03:17 -0400 Received: from localhost (unknown [127.0.0.1]) by smtp.cica.es (Postfix) with ESMTP id F3B9B51EF77; Tue, 3 Jun 2014 07:03:15 +0000 (UTC) X-Virus-Scanned: amavisd-new at cica.es Received: from smtp.cica.es ([127.0.0.1]) by localhost (mail.cica.es [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kYyJTWTLrIzX; Tue, 3 Jun 2014 09:03:06 +0200 (CEST) Received: from nfdev.cica.es (nfdev.cica.es [IPv6:2a00:9ac0:c1ca:31::220]) by smtp.cica.es (Postfix) with ESMTP id B033C51EF72; Tue, 3 Jun 2014 09:03:06 +0200 (CEST) Subject: [nf_tables PATCH v2 2/2] netfilter: nf_tables: split nft_log in AF-specific modules To: netfilter-devel@vger.kernel.org From: Arturo Borrero Gonzalez Cc: pablo@netfilter.org Date: Tue, 03 Jun 2014 09:03:05 +0200 Message-ID: <20140603070201.2754.47078.stgit@nfdev.cica.es> In-Reply-To: <20140603070137.2754.21930.stgit@nfdev.cica.es> References: <20140603070137.2754.21930.stgit@nfdev.cica.es> User-Agent: StGit/0.15 MIME-Version: 1.0 Sender: netfilter-devel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netfilter-devel@vger.kernel.org This patch split the nft_log module in AF-specific modules. For NFPROTO_INET, it does nothing but dispatch to the AF-specific modules. The main benefit of this patch is that we can now use nft_log from userspace without being force to modprobe ipt_LOG, which was an annoying bug. Some new symbols are added to Kconfig: NFT_LOG_IPV4, NFT_LOG_IPV6 and NFT_LOG_INET. Signed-off-by: Arturo Borrero Gonzalez --- v2: actually solve the nft_log issue regarding the need to load ipt_LOG. include/net/netfilter/ipv4/nft_log_ipv4.h | 8 ++ include/net/netfilter/ipv6/nft_log_ipv6.h | 8 ++ include/net/netfilter/nft_log.h | 16 ++++ net/ipv4/netfilter/Kconfig | 5 + net/ipv4/netfilter/Makefile | 1 net/ipv4/netfilter/nft_log_ipv4.c | 95 ++++++++++++++++++++++++ net/ipv6/netfilter/Kconfig | 5 + net/ipv6/netfilter/Makefile | 1 net/ipv6/netfilter/nft_log_ipv6.c | 95 ++++++++++++++++++++++++ net/netfilter/Kconfig | 5 + net/netfilter/Makefile | 1 net/netfilter/nft_log.c | 65 +++------------- net/netfilter/nft_log_inet.c | 116 +++++++++++++++++++++++++++++ 13 files changed, 366 insertions(+), 55 deletions(-) create mode 100644 include/net/netfilter/ipv4/nft_log_ipv4.h create mode 100644 include/net/netfilter/ipv6/nft_log_ipv6.h create mode 100644 include/net/netfilter/nft_log.h create mode 100644 net/ipv4/netfilter/nft_log_ipv4.c create mode 100644 net/ipv6/netfilter/nft_log_ipv6.c create mode 100644 net/netfilter/nft_log_inet.c -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/include/net/netfilter/ipv4/nft_log_ipv4.h b/include/net/netfilter/ipv4/nft_log_ipv4.h new file mode 100644 index 0000000..031eabb --- /dev/null +++ b/include/net/netfilter/ipv4/nft_log_ipv4.h @@ -0,0 +1,8 @@ +#ifndef _NFT_LOG_IPV4_H_ +#define _NFT_LOG_IPV4_H_ + +void nft_log_ipv4_eval(const struct nft_expr *expr, + struct nft_data data[NFT_REG_MAX + 1], + const struct nft_pktinfo *pkt); + +#endif /* _NFT_LOG_IPV4_H_ */ diff --git a/include/net/netfilter/ipv6/nft_log_ipv6.h b/include/net/netfilter/ipv6/nft_log_ipv6.h new file mode 100644 index 0000000..8ed9cd7 --- /dev/null +++ b/include/net/netfilter/ipv6/nft_log_ipv6.h @@ -0,0 +1,8 @@ +#ifndef _NFT_LOG_IPV6_H_ +#define _NFT_LOG_IPV6_H_ + +void nft_log_ipv6_eval(const struct nft_expr *expr, + struct nft_data data[NFT_REG_MAX + 1], + const struct nft_pktinfo *pkt); + +#endif /* _NFT_LOG_IPV6_H_ */ diff --git a/include/net/netfilter/nft_log.h b/include/net/netfilter/nft_log.h new file mode 100644 index 0000000..818c289 --- /dev/null +++ b/include/net/netfilter/nft_log.h @@ -0,0 +1,16 @@ +#ifndef _NFT_LOG_H_ +#define _NFT_LOG_H_ + +struct nft_log { + struct nf_loginfo loginfo; + char *prefix; +}; + +extern const struct nla_policy nft_log_policy[]; + +int nft_log_init(const struct nft_ctx *ctx, const struct nft_expr *expr, + const struct nlattr * const tb[]); +void nft_log_destroy(const struct nft_ctx *ctx, const struct nft_expr *expr); +int nft_log_dump(struct sk_buff *skb, const struct nft_expr *expr); + +#endif /* _NFT_LOG_H_ */ diff --git a/net/ipv4/netfilter/Kconfig b/net/ipv4/netfilter/Kconfig index e09f364..db32b73 100644 --- a/net/ipv4/netfilter/Kconfig +++ b/net/ipv4/netfilter/Kconfig @@ -66,6 +66,11 @@ config NFT_REJECT_IPV4 default NFT_REJECT tristate +config NFT_LOG_IPV4 + depends on NF_TABLES_IPV4 + default NFT_LOG + tristate + config NF_TABLES_ARP depends on NF_TABLES tristate "ARP nf_tables support" diff --git a/net/ipv4/netfilter/Makefile b/net/ipv4/netfilter/Makefile index d2f4b29..1f153e5 100644 --- a/net/ipv4/netfilter/Makefile +++ b/net/ipv4/netfilter/Makefile @@ -31,6 +31,7 @@ obj-$(CONFIG_NF_TABLES_IPV4) += nf_tables_ipv4.o obj-$(CONFIG_NFT_CHAIN_ROUTE_IPV4) += nft_chain_route_ipv4.o obj-$(CONFIG_NFT_CHAIN_NAT_IPV4) += nft_chain_nat_ipv4.o obj-$(CONFIG_NFT_REJECT_IPV4) += nft_reject_ipv4.o +obj-$(CONFIG_NFT_LOG_IPV4) += nft_log_ipv4.o obj-$(CONFIG_NF_TABLES_ARP) += nf_tables_arp.o # generic IP tables diff --git a/net/ipv4/netfilter/nft_log_ipv4.c b/net/ipv4/netfilter/nft_log_ipv4.c new file mode 100644 index 0000000..9ec82d1 --- /dev/null +++ b/net/ipv4/netfilter/nft_log_ipv4.c @@ -0,0 +1,95 @@ +/* + * Copyright (c) 2008-2009 Patrick McHardy + * Copyright (c) 2014 Arturo Borrero Gonzalez + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +void nft_log_ipv4_eval(const struct nft_expr *expr, + struct nft_data data[NFT_REG_MAX + 1], + const struct nft_pktinfo *pkt) +{ + const struct nft_log *priv = nft_expr_priv(expr); + struct net *net = dev_net(pkt->in ? pkt->in : pkt->out); + + nf_log_ip_packet(net, pkt->ops->pf, pkt->ops->hooknum, pkt->skb, + pkt->in, pkt->out, &priv->loginfo, priv->prefix); +} +EXPORT_SYMBOL_GPL(nft_log_ipv4_eval); + +static struct nft_expr_type nft_log_ipv4_type; +static const struct nft_expr_ops nft_log_ipv4_ops = { + .type = &nft_log_ipv4_type, + .size = NFT_EXPR_SIZE(sizeof(struct nft_log)), + .eval = nft_log_ipv4_eval, + .init = nft_log_init, + .destroy = nft_log_destroy, + .dump = nft_log_dump, +}; + +static struct nft_expr_type nft_log_ipv4_type __read_mostly = { + .family = NFPROTO_IPV4, + .name = "log", + .ops = &nft_log_ipv4_ops, + .policy = nft_log_policy, + .maxattr = NFTA_LOG_MAX, + .owner = THIS_MODULE, +}; + +static struct nf_logger nft_log_ipv4_logger __read_mostly = { + .name = "nft_log_ipv4", + .logfn = &nf_log_ip_packet, + .me = THIS_MODULE, +}; + +static int __net_init nft_log_ipv4_init_net(struct net *net) +{ + nf_log_set(net, NFPROTO_IPV4, &nft_log_ipv4_logger); +} + +static void __net_exit nft_log_ipv4_exit_net(struct net *net) +{ + nf_log_unset(net, &nft_log_ipv4_logger); +} + +static struct pernet_operations nft_log_ipv4_net_ops = { + .init = nft_log_ipv4_init_net, + .exit = nft_log_ipv4_exit_net, +}; + +static int __init nft_log_ipv4_module_init(void) +{ + int ret; + + ret = register_pernet_subsys(&nft_log_ipv4_net_ops); + if (ret < 0) + return ret; + + return nft_register_expr(&nft_log_ipv4_type); +} + +static void __exit nft_log_ipv4_module_exit(void) +{ + unregister_pernet_subsys(&nft_log_ipv4_net_ops); + nft_unregister_expr(&nft_log_ipv4_type); +} + +module_init(nft_log_ipv4_module_init); +module_exit(nft_log_ipv4_module_exit); + +MODULE_LICENSE("GPL"); +MODULE_AUTHOR("Patrick McHardy "); +MODULE_ALIAS_NFT_AF_EXPR(AF_INET, "log"); diff --git a/net/ipv6/netfilter/Kconfig b/net/ipv6/netfilter/Kconfig index 4101334..4ee6aa3 100644 --- a/net/ipv6/netfilter/Kconfig +++ b/net/ipv6/netfilter/Kconfig @@ -55,6 +55,11 @@ config NFT_REJECT_IPV6 default NFT_REJECT tristate +config NFT_LOG_IPV6 + depends on NF_TABLES_IPV6 + default NFT_LOG + tristate + config IP6_NF_IPTABLES tristate "IP6 tables support (required for filtering)" depends on INET && IPV6 diff --git a/net/ipv6/netfilter/Makefile b/net/ipv6/netfilter/Makefile index 4a5ffc6..047126d 100644 --- a/net/ipv6/netfilter/Makefile +++ b/net/ipv6/netfilter/Makefile @@ -28,6 +28,7 @@ obj-$(CONFIG_NF_TABLES_IPV6) += nf_tables_ipv6.o obj-$(CONFIG_NFT_CHAIN_ROUTE_IPV6) += nft_chain_route_ipv6.o obj-$(CONFIG_NFT_CHAIN_NAT_IPV6) += nft_chain_nat_ipv6.o obj-$(CONFIG_NFT_REJECT_IPV6) += nft_reject_ipv6.o +obj-$(CONFIG_NFT_LOG_IPV6) += nft_log_ipv6.o # matches obj-$(CONFIG_IP6_NF_MATCH_AH) += ip6t_ah.o diff --git a/net/ipv6/netfilter/nft_log_ipv6.c b/net/ipv6/netfilter/nft_log_ipv6.c new file mode 100644 index 0000000..3844ca5 --- /dev/null +++ b/net/ipv6/netfilter/nft_log_ipv6.c @@ -0,0 +1,95 @@ +/* + * Copyright (c) 2008-2009 Patrick McHardy + * Copyright (c) 2014 Arturo Borrero Gonzalez + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +void nft_log_ipv6_eval(const struct nft_expr *expr, + struct nft_data data[NFT_REG_MAX + 1], + const struct nft_pktinfo *pkt) +{ + const struct nft_log *priv = nft_expr_priv(expr); + struct net *net = dev_net(pkt->in ? pkt->in : pkt->out); + + nf_log_ip6_packet(net, pkt->ops->pf, pkt->ops->hooknum, pkt->skb, + pkt->in, pkt->out, &priv->loginfo, priv->prefix); +} +EXPORT_SYMBOL_GPL(nft_log_ipv6_eval); + +static struct nft_expr_type nft_log_ipv6_type; +static const struct nft_expr_ops nft_log_ipv6_ops = { + .type = &nft_log_ipv6_type, + .size = NFT_EXPR_SIZE(sizeof(struct nft_log)), + .eval = nft_log_ipv6_eval, + .init = nft_log_init, + .destroy = nft_log_destroy, + .dump = nft_log_dump, +}; + +static struct nft_expr_type nft_log_ipv6_type __read_mostly = { + .family = NFPROTO_IPV6, + .name = "log", + .ops = &nft_log_ipv6_ops, + .policy = nft_log_policy, + .maxattr = NFTA_LOG_MAX, + .owner = THIS_MODULE, +}; + +static struct nf_logger nft_log_ipv6_logger __read_mostly = { + .name = "nft_log_ipv6", + .logfn = &nf_log_ip6_packet, + .me = THIS_MODULE, +}; + +static int __net_init nft_log_ipv6_init_net(struct net *net) +{ + nf_log_set(net, NFPROTO_IPV6, &nft_log_ipv6_logger); +} + +static void __net_exit nft_log_ipv6_exit_net(struct net *net) +{ + nf_log_unset(net, &nft_log_ipv6_logger); +} + +static struct pernet_operations nft_log_ipv6_net_ops = { + .init = nft_log_ipv6_init_net, + .exit = nft_log_ipv6_exit_net, +}; + +static int __init nft_log_ipv6_module_init(void) +{ + int ret; + + ret = register_pernet_subsys(&nft_log_ipv6_net_ops); + if (ret < 0) + return ret; + + return nft_register_expr(&nft_log_ipv6_type); +} + +static void __exit nft_log_ipv6_module_exit(void) +{ + unregister_pernet_subsys(&nft_log_ipv6_net_ops); + nft_unregister_expr(&nft_log_ipv6_type); +} + +module_init(nft_log_ipv6_module_init); +module_exit(nft_log_ipv6_module_exit); + +MODULE_LICENSE("GPL"); +MODULE_AUTHOR("Patrick McHardy "); +MODULE_ALIAS_NFT_AF_EXPR(AF_INET6, "log"); diff --git a/net/netfilter/Kconfig b/net/netfilter/Kconfig index 4133172..701f39d 100644 --- a/net/netfilter/Kconfig +++ b/net/netfilter/Kconfig @@ -532,6 +532,11 @@ config NFT_REJECT_INET default NFT_REJECT tristate +config NFT_LOG_INET + depends on NF_TABLES_INET + default NFT_LOG + tristate + config NFT_COMPAT depends on NF_TABLES depends on NETFILTER_XTABLES diff --git a/net/netfilter/Makefile b/net/netfilter/Makefile index d19e961..5ac6480 100644 --- a/net/netfilter/Makefile +++ b/net/netfilter/Makefile @@ -87,6 +87,7 @@ obj-$(CONFIG_NFT_RBTREE) += nft_rbtree.o obj-$(CONFIG_NFT_HASH) += nft_hash.o obj-$(CONFIG_NFT_COUNTER) += nft_counter.o obj-$(CONFIG_NFT_LOG) += nft_log.o +obj-$(CONFIG_NFT_LOG_INET) += nft_log_inet.o # generic X tables obj-$(CONFIG_NETFILTER_XTABLES) += x_tables.o xt_tcpudp.o diff --git a/net/netfilter/nft_log.c b/net/netfilter/nft_log.c index 10cfb15..6f2f541 100644 --- a/net/netfilter/nft_log.c +++ b/net/netfilter/nft_log.c @@ -17,35 +17,20 @@ #include #include #include +#include static const char *nft_log_null_prefix = ""; -struct nft_log { - struct nf_loginfo loginfo; - char *prefix; -}; - -static void nft_log_eval(const struct nft_expr *expr, - struct nft_data data[NFT_REG_MAX + 1], - const struct nft_pktinfo *pkt) -{ - const struct nft_log *priv = nft_expr_priv(expr); - struct net *net = dev_net(pkt->in ? pkt->in : pkt->out); - - nf_log_packet(net, pkt->ops->pf, pkt->ops->hooknum, pkt->skb, pkt->in, - pkt->out, &priv->loginfo, "%s", priv->prefix); -} - -static const struct nla_policy nft_log_policy[NFTA_LOG_MAX + 1] = { +const struct nla_policy nft_log_policy[NFTA_LOG_MAX + 1] = { [NFTA_LOG_GROUP] = { .type = NLA_U16 }, [NFTA_LOG_PREFIX] = { .type = NLA_STRING }, [NFTA_LOG_SNAPLEN] = { .type = NLA_U32 }, [NFTA_LOG_QTHRESHOLD] = { .type = NLA_U16 }, }; +EXPORT_SYMBOL_GPL(nft_log_policy); -static int nft_log_init(const struct nft_ctx *ctx, - const struct nft_expr *expr, - const struct nlattr * const tb[]) +int nft_log_init(const struct nft_ctx *ctx, const struct nft_expr *expr, + const struct nlattr * const tb[]) { struct nft_log *priv = nft_expr_priv(expr); struct nf_loginfo *li = &priv->loginfo; @@ -73,17 +58,18 @@ static int nft_log_init(const struct nft_ctx *ctx, return 0; } +EXPORT_SYMBOL_GPL(nft_log_init); -static void nft_log_destroy(const struct nft_ctx *ctx, - const struct nft_expr *expr) +void nft_log_destroy(const struct nft_ctx *ctx, const struct nft_expr *expr) { struct nft_log *priv = nft_expr_priv(expr); if (priv->prefix != nft_log_null_prefix) kfree(priv->prefix); } +EXPORT_SYMBOL_GPL(nft_log_destroy); -static int nft_log_dump(struct sk_buff *skb, const struct nft_expr *expr) +int nft_log_dump(struct sk_buff *skb, const struct nft_expr *expr) { const struct nft_log *priv = nft_expr_priv(expr); const struct nf_loginfo *li = &priv->loginfo; @@ -107,38 +93,7 @@ static int nft_log_dump(struct sk_buff *skb, const struct nft_expr *expr) nla_put_failure: return -1; } - -static struct nft_expr_type nft_log_type; -static const struct nft_expr_ops nft_log_ops = { - .type = &nft_log_type, - .size = NFT_EXPR_SIZE(sizeof(struct nft_log)), - .eval = nft_log_eval, - .init = nft_log_init, - .destroy = nft_log_destroy, - .dump = nft_log_dump, -}; - -static struct nft_expr_type nft_log_type __read_mostly = { - .name = "log", - .ops = &nft_log_ops, - .policy = nft_log_policy, - .maxattr = NFTA_LOG_MAX, - .owner = THIS_MODULE, -}; - -static int __init nft_log_module_init(void) -{ - return nft_register_expr(&nft_log_type); -} - -static void __exit nft_log_module_exit(void) -{ - nft_unregister_expr(&nft_log_type); -} - -module_init(nft_log_module_init); -module_exit(nft_log_module_exit); +EXPORT_SYMBOL_GPL(nft_log_dump); MODULE_LICENSE("GPL"); MODULE_AUTHOR("Patrick McHardy "); -MODULE_ALIAS_NFT_EXPR("log"); diff --git a/net/netfilter/nft_log_inet.c b/net/netfilter/nft_log_inet.c new file mode 100644 index 0000000..2394532 --- /dev/null +++ b/net/netfilter/nft_log_inet.c @@ -0,0 +1,116 @@ +/* + * Copyright (c) 2008-2009 Patrick McHardy + * Copyright (c) 2014 Arturo Borrero Gonzalez + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +static void nft_log_inet_eval(const struct nft_expr *expr, + struct nft_data data[NFT_REG_MAX + 1], + const struct nft_pktinfo *pkt) +{ + switch (pkt->ops->pf) { + case NFPROTO_IPV4: + nft_log_ipv4_eval(expr, data, pkt); + break; + case NFPROTO_IPV6: + nft_log_ipv6_eval(expr, data, pkt); + break; + } +} + +static struct nft_expr_type nft_log_inet_type; +static const struct nft_expr_ops nft_log_inet_ops = { + .type = &nft_log_inet_type, + .size = NFT_EXPR_SIZE(sizeof(struct nft_log)), + .eval = nft_log_inet_eval, + .init = nft_log_init, + .destroy = nft_log_destroy, + .dump = nft_log_dump, +}; + +static struct nft_expr_type nft_log_inet_type __read_mostly = { + .family = NFPROTO_INET, + .name = "log", + .ops = &nft_log_inet_ops, + .policy = nft_log_policy, + .maxattr = NFTA_LOG_MAX, + .owner = THIS_MODULE, +}; + +static struct nf_logger nft_log_inet_ipv4_logger __read_mostly = { + .name = "nft_log_inet_ipv4", + .logfn = &nf_log_ip_packet, + .me = THIS_MODULE, +}; + +#if IS_ENABLED(CONFIG_IP6_NF_IPTABLES) +static struct nf_logger nft_log_inet_ipv6_logger __read_mostly = { + .name = "nft_log_inet_ipv6", + .logfn = &nf_log_ip6_packet, + .me = THIS_MODULE, +}; +#endif + +static void __net_init nft_log_inet_init_net(struct net *net) +{ + nf_log_set(net, NFPROTO_IPV4, &nft_log_inet_ipv4_logger); +#if IS_ENABLED(CONFIG_IP6_NF_IPTABLES) + nf_log_set(net, NFPROTO_IPV6, &nft_log_inet_ipv6_logger); +#endif +} + +static void __net_exit nft_log_inet_exit_net(struct net *net) +{ + nf_log_unset(net, &nft_log_inet_ipv4_logger); +#if IS_ENABLED(CONFIG_IP6_NF_IPTABLES) + nf_log_unset(net, &nft_log_inet_ipv6_logger); +#endif +} + +static struct pernet_operations nft_log_inet_net_ops = { + .init = nft_log_inet_init_net, + .exit = nft_log_inet_exit_net, +}; + +static int __init nft_log_inet_module_init(void) +{ + int ret; + + ret = register_pernet_subsys(&nft_log_inet_net_ops); + if (ret < 0) + return ret; + + return nft_register_expr(&nft_log_inet_type); +} + +static void __exit nft_log_inet_module_exit(void) +{ + unregister_pernet_subsys(&nft_log_inet_net_ops); + nft_unregister_expr(&nft_log_inet_type); +} + +module_init(nft_log_inet_module_init); +module_exit(nft_log_inet_module_exit); + +MODULE_LICENSE("GPL"); +MODULE_AUTHOR("Patrick McHardy "); +MODULE_ALIAS_NFT_AF_EXPR(1, "log");