Add userspace component of the TPROXY target
PATCH 1/2 Add userspace component of the TPROXY target
From: Balazs Scheidler <[email protected]>
---
extensions/libxt_TPROXY.c | 155 +++++++++++++++++++++++++++++++++++ extensions/libxt_TPROXY.man | 21 +++++ include/linux/netfilter/xt_TPROXY.h | 14 +++ 3 files changed, 190 insertions(+), 0 deletions(-) create mode 100644 extensions/libxt_TPROXY.c create mode 100644 extensions/libxt_TPROXY.man create mode 100644 include/linux/netfilter/xt_TPROXY.h
extensions/libxt_TPROXY.c new file +155 -0
@@ -0,0 +1,155 @@ +/* +* Shared library add-on to iptables to add TPROXY target support. +* +* Copyright (C) 2002-2007 BalaBit IT Ltd. +*/ +#include <getopt.h> +#include <stdbool.h> +#include <stdio.h> +#include <string.h> +#include <stdlib.h> +#include <limits.h> + +#include <iptables.h> +#include <xtables.h> +#include <linux/netfilter/x_tables.h> +#include <linux/netfilter/xt_TPROXY.h> + +static const struct option tproxy_opts[] = { + {"on-port" , true,NULL ,'1' }, + {"on-ip" , true,NULL ,'2' }, + {"tproxy-mark" , true,NULL ,'3' }, + {NULL }, +}; + +#define PARAM_ONPORT 1 +#define PARAM_ONIP 2 +#define PARAM_MARK 4 + +static void tproxy_help (void ) +{ +printf ( +"TPROXY target v%s options:\n" +" --on-port port Redirect connection to port, or the original port if 0\n" +" --on-ip ip Optionally redirect to the given IP\n" +" --tproxy-mark value/mask Mark packets with the given value/mask\n\n" , +XTABLES_VERSION ); +} + +static void parse_tproxy_lport (const char *s,struct xt_tproxy_target_info *info) +{ +unsigned int lport; + +if (string_to_number (s,0 ,65535 , &lport) !=-1 ) + info->lport =htons (lport); +else +exit_error (PARAMETER_PROBLEM ,"bad --on-port \"%s\"" , s); +} + +static void parse_tproxy_laddr (const char *s,struct xt_tproxy_target_info *info) +{ +struct in_addr *laddr; + +if ((laddr =numeric_to_ipaddr (s)) ==NULL ) +exit_error (PARAMETER_PROBLEM ,"bad --on-ip \"%s\"" , s); + info->laddr = laddr->s_addr; +} + +static void parse_tproxy_mark (char *s,struct xt_tproxy_target_info *info) +{ +unsigned long tmp; +char *slash; + + slash =strchr (s,'/' ); + info->mark_mask = (u_int32_t) -1 ; +if (slash !=NULL ) { + *slash ='\0' ; +if (string_to_number_l (slash +1 ,0 ,ULONG_MAX , &tmp) <0 ) +exit_error (PARAMETER_PROBLEM , +"bad mask in --tproxy-mark \"%s\"" , s); + info->mark_mask = tmp; + } +if (string_to_number_l (s,0 ,ULONG_MAX , &tmp) <0 ) +exit_error (PARAMETER_PROBLEM , +"bad value in --tproxy-mark \"%s\"" , s); + info->mark_value = tmp; +} + +static int tproxy_parse (int c,char **argv,int invert,unsigned int *flags, +const void *entry,struct xt_entry_target **target) +{ +struct xt_tproxy_target_info *tproxyinfo = (void *)(*target)->data; + +switch (c) { +case '1' : +if (*flags &PARAM_ONPORT ) +exit_error (PARAMETER_PROBLEM , +"TPROXY target: Can't specify --on-port twice" ); +parse_tproxy_lport (optarg, tproxyinfo); + *flags |=PARAM_ONPORT ; +return 1 ; +case '2' : +if (*flags &PARAM_ONIP ) +exit_error (PARAMETER_PROBLEM , +"TPROXY target: Can't specify --on-ip twice" ); +parse_tproxy_laddr (optarg, tproxyinfo); + *flags |=PARAM_ONIP ; +return 1 ; +case '3' : +if (*flags &PARAM_MARK ) +exit_error (PARAMETER_PROBLEM , +"TPROXY target: Can't specify --tproxy-mark twice" ); +parse_tproxy_mark (optarg, tproxyinfo); + *flags |=PARAM_MARK ; +return 1 ; + } + +return 0 ; +} + +static void tproxy_check (unsigned int flags) +{ +if (!(flags &PARAM_ONPORT )) +exit_error (PARAMETER_PROBLEM , +"TPROXY target: Parameter --on-port is required" ); +} + +static void tproxy_print (const void *ip,const struct xt_entry_target *target, +int numeric) +{ +const struct xt_tproxy_target_info *info = (const void *)target->data; +printf ("TPROXY redirect %s:%u mark 0x%x/0x%x" , +ipaddr_to_numeric ((const struct in_addr *)&info->laddr), +ntohs (info->lport), (unsigned int )info->mark_value, + (unsigned int )info->mark_mask); +} + +static void tproxy_save (const void *ip,const struct xt_entry_target *target) +{ +const struct xt_tproxy_target_info *info = (const void *)target->data; + +printf ("--on-port %u " ,ntohs (info->lport)); +printf ("--on-ip %s " , +ipaddr_to_numeric ((const struct in_addr *)&info->laddr)); +printf ("--tproxy-mark 0x%x/0x%x " , + (unsigned int )info->mark_value, (unsigned int )info->mark_mask); +} + +static struct xtables_target tproxy_reg = { + .name ="TPROXY" , + .family =AF_INET , + .version =XTABLES_VERSION , + .size =XT_ALIGN (sizeof (struct xt_tproxy_target_info )), + .userspacesize =XT_ALIGN (sizeof (struct xt_tproxy_target_info )), + .help = tproxy_help, + .parse = tproxy_parse, + .final_check = tproxy_check, + .print = tproxy_print, + .save = tproxy_save, + .extra_opts = tproxy_opts, +}; + +void _init (void ) +{ +xtables_register_target (&tproxy_reg); +}
extensions/libxt_TPROXY.man new file +21 -0
@@ -0,0 +1,21 @@ +This target is only valid in the \fBmangle\fR table, in the \fBPREROUTING\fR +chain and user-defined chains which are only called from this chain. It +redirects the packet to a local socket without changing the packet header in +any way. It can also change the mark value which can then be used in advanced +routing rules. +It takes three options: +.TP +\fB--on-port\fR \fIport\fR +This specifies a destination port to use. It is a required option, 0 means the +new destination port is the same as the original. This is only valid if the +rule also specifies \fB-p tcp\fR or \fB-p udp\fR. +.TP +\fB--on-ip\fR \fIaddress\fR +This specifies a destination address to use. By default the address is the IP +address of the incoming interface. This is only valid if the rule also +specifies \fB-p tcp\fR or \fR-p udp\fR. +.TP +\fB--tproxy-mark\fR \fIvalue\fR[\fB/\fR\fImask\fR] +Marks packets with the given value/mask. The fwmark value set here can be used +by advanced routing. (Required for transparent proxying to work: otherwise +these packets will get forwarded, which is probably not what you want.)
include/linux/netfilter/xt_TPROXY.h new file +14 -0
@@ -0,0 +1,14 @@ +#ifndef _XT_TPROXY_H_target +#define _XT_TPROXY_H_target + +/* TPROXY target is capable of marking the packet to perform +* redirection. We can get rid of that whenever we get support for +* mutliple targets in the same rule. */ +struct xt_tproxy_target_info { +u_int32_t mark_mask; +u_int32_t mark_value; +__be32 laddr; +__be16 lport; +}; + +#endif /* _XT_TPROXY_H_target */
PATCH 2/2 Add userspace component of the socket match
From: Balazs Scheidler <[email protected]>
---
extensions/libxt_socket.c | 39 +++++++++++++++++++++++++++++++++++++++ extensions/libxt_socket.man | 2 ++ 2 files changed, 41 insertions(+), 0 deletions(-) create mode 100644 extensions/libxt_socket.c create mode 100644 extensions/libxt_socket.man
extensions/libxt_socket.c new file +39 -0
@@ -0,0 +1,39 @@ +/* +* Shared library add-on to iptables to add early socket matching support. +* +* Copyright (C) 2007 BalaBit IT Ltd. +*/ +#include <stdio.h> +#include <getopt.h> +#include <iptables.h> + +static void socket_help (void ) +{ +printf ("socket v%s has no options\n\n" ,XTABLES_VERSION ); +} + +static int socket_parse (int c,char **argv,int invert,unsigned int *flags, +const void *entry,struct xt_entry_match **match) +{ +return 0 ; +} + +static void socket_check (unsigned int flags) +{ +} + +static struct xtables_match socket_reg = { + .name ="socket" , + .version =XTABLES_VERSION , + .family =AF_INET , + .size =XT_ALIGN (0 ), + .userspacesize =XT_ALIGN (0 ), + .parse = socket_parse, + .final_check = socket_check, + .help = socket_help, +}; + +void _init (void ) +{ +xtables_register_match (&socket_reg); +}
extensions/libxt_socket.man new file +2 -0
@@ -0,0 +1,2 @@ +This matches if an open socket can be found by doing a socket lookup on the +packet.
Re: [iptables PATCH 2/2] [TPROXY] Add userspace component of the socket match
Quoted 30 lines · KOVACS Krisztian
extensions/libxt_socket.c new file +29
-0@@ -0,0 +1,39 @@ +/* +* Shared library add-on to iptables to add early socket matching support. +* +* Copyright (C) 2007 BalaBit IT Ltd. +*/ +#include <stdio.h> +#include <getopt.h> +#include <iptables.h> + +static void socket_help (void ) +{ +printf ("socket v%s has no options\n\n" ,XTABLES_VERSION ); +} + +static int socket_parse (int c,char **argv,int invert,unsigned int *flags, +const void *entry,struct xt_entry_match **match) +{ +return 0 ; +} + +static void socket_check (unsigned int flags) +{ +} + +static struct xtables_match socket_reg = { + .name ="socket" , + .version =XTABLES_VERSION , + .family =AF_INET , + .size =XT_ALIGN (0 ),
extensions/libxt_socket.c line 39
Quoted 10 lines
extensions/libxt_socket.c new file +10
-0+ .userspacesize = XT_ALIGN(0), + .parse = socket_parse, + .final_check = socket_check, + .help = socket_help, +}; + +void _init (void ) +{ +xtables_register_match (&socket_reg); +}
I doubt this trivial code is anything you could copyright.
extensions/libxt_socket.man line 2
Quoted 5 lines
extensions/libxt_socket.man new file +2
-0@@ -0,0 +1,2 @@ +This matches if an open socket can be found by doing a socket lookup on the +packet.
I think, while at it, this match should be extended by options to find only listening sockets. People sometimes want something like this because they cannot possibly know the used port numbers in advance and do not want to use port ranges that could possible be too broad (--dport x:y) that would accidentally match other connections.
Re: [iptables PATCH 2/2] [TPROXY] Add userspace component of the socket match
Quoted 10 lines · Jan Engelhardt
extensions/libxt_socket.c line 6
Quoted 7 lines · KOVACS Krisztian
extensions/libxt_socket.c new file +6
-0@@ -0,0 +1,39 @@ +/* +* Shared library add-on to iptables to add early socket matching support. +* +* Copyright (C) 2007 BalaBit IT Ltd. +*/ + [...]I doubt this trivial code is anything you could copyright.
It doesn't hurt to claim copyright though. No need to change this.
extensions/libxt_TPROXY.c line 23
Quoted 6 lines · KOVACS Krisztian
extensions/libxt_TPROXY.c new file +6
-0+static const struct option tproxy_opts[] = { + {"on-port" , true,NULL ,'1' }, + {"on-ip" , true,NULL ,'2' }, + {"tproxy-mark" , true,NULL ,'3' }, + {NULL }, +};
C99 init preferred :)
extensions/libxt_TPROXY.c line 37
Quoted 9 lines
extensions/libxt_TPROXY.c new file +9
-0+static void tproxy_help (void ) +{ +printf ( +"TPROXY target v%s options:\n" +" --on-port port Redirect connection to port, or the original port if 0\n" +" --on-ip ip Optionally redirect to the given IP\n" +" --tproxy-mark value/mask Mark packets with the given value/mask\n\n" , +XTABLES_VERSION ); +}
Omit XTABLES_VERSION, it is hardly helpful here, because the TPROXY extension is unlikely to change and as such, giving it a version number that always increases as iptables releases are done seems kinda blunt.
You also probably want to tell the user "value[/mask]", because the code indicates it is optional.
Quoted 9 lines
extensions/libxt_TPROXY.c new file +9
-0+static void parse_tproxy_mark (char *s,struct xt_tproxy_target_info *info) +{ +unsigned long tmp; +char *slash; + + slash =strchr (s,'/' ); + info->mark_mask = (u_int32_t) -1 ; +if (slash !=NULL ) { + *slash ='\0' ;
extensions/libxt_TPROXY.c line 76
Quoted 10 lines
extensions/libxt_TPROXY.c new file +10
-0+if (string_to_number_l (slash +1 ,0 ,ULONG_MAX , &tmp) <0 ) +exit_error (PARAMETER_PROBLEM , +"bad mask in --tproxy-mark \"%s\"" , s); + info->mark_mask = tmp; + } +if (string_to_number_l (s,0 ,ULONG_MAX , &tmp) <0 ) +exit_error (PARAMETER_PROBLEM , +"bad value in --tproxy-mark \"%s\"" , s); + info->mark_value = tmp; +}
ULONG_MAX is wrong here, as it is 2^64-1 on x86_64, but mark_mask is only 32-bit.
static void tproxy_tg_init(...)
{
info->mark_mask = ~0U;
}static void parse_tproxy_mark(...)
{
unsigned int v;
char *end;
if (!strtonum(optarg, &end, &v, 0, UINT_MAX))
exit_error(problem);
info->mark_value = v;
if (*end == '\0')
return;
if (*end != '/')
exit_error(syntax_problem)
if (!strtonum(optarg, NULL, &v, 0, UINT_MAX))
exit_error(problem);
info->mark_mask = v;
}Manpage looks good.
Quoted 1 line · KOVACS Krisztian
From: Balazs Scheidler <[email protected]>
Could I get a Signed-off-by: line for these patches please? Thanks!
Hi,
Quoted 5 lines · Patrick McHardy
Quoted 1 line · KOVACS Krisztian
From: Balazs Scheidler <[email protected]>
Could I get a Signed-off-by: line for these patches please? Thanks!
I'm going to resend the patches with the problems spotted by Jan fixed and also with a proper signed-off-by line.
Signature · 1 line
KOVACS Krisztian