* [PATCH 0/3] IPv6 tarpit support
@ 2012-07-07 21:20 Josh Hunt
2012-07-07 21:20 ` [PATCH 1/3] xtables: tarpit: Make tarpit code generic Josh Hunt
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Josh Hunt @ 2012-07-07 21:20 UTC (permalink / raw)
To: netfilter-devel; +Cc: Josh Hunt
First attempt at adding IPv6 support for TARPIT target. Tried to follow the v4
implementation where applicable. Also borrowed some code from ip6t_REJECT.c.
Basic functionality has been tested for v4 and v6 in all 3 modes. Everything
appears to be working correctly.
Josh
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/3] xtables: tarpit: Make tarpit code generic
2012-07-07 21:20 [PATCH 0/3] IPv6 tarpit support Josh Hunt
@ 2012-07-07 21:20 ` Josh Hunt
2012-07-08 12:58 ` Jan Engelhardt
2012-07-07 21:21 ` [PATCH 2/3] xtables: tarpit: Add IPv6 support Josh Hunt
2012-07-07 21:21 ` [PATCH 3/3] xtables: libxt_TARPIT: Enable " Josh Hunt
2 siblings, 1 reply; 6+ messages in thread
From: Josh Hunt @ 2012-07-07 21:20 UTC (permalink / raw)
To: netfilter-devel; +Cc: Josh Hunt
Moves TCP header manipulation into a separate function to allow code-sharing for
IPv6 support.
Signed-off-by: Josh Hunt <johunt@akamai.com>
---
extensions/xt_TARPIT.c | 133 +++++++++++++++++++++++++-----------------------
1 files changed, 69 insertions(+), 64 deletions(-)
diff --git a/extensions/xt_TARPIT.c b/extensions/xt_TARPIT.c
index db24f90..9a09e75 100644
--- a/extensions/xt_TARPIT.c
+++ b/extensions/xt_TARPIT.c
@@ -51,70 +51,7 @@
#include "compat_xtables.h"
#include "xt_TARPIT.h"
-static void tarpit_tcp(struct sk_buff *oldskb, unsigned int hook,
- unsigned int mode)
-{
- struct tcphdr _otcph, *oth, *tcph;
- unsigned int addr_type = RTN_UNSPEC;
- struct sk_buff *nskb;
- const struct iphdr *oldhdr;
- struct iphdr *niph;
- uint16_t tmp, payload;
-
- /* A truncated TCP header is not going to be useful */
- if (oldskb->len < ip_hdrlen(oldskb) + sizeof(struct tcphdr))
- return;
-
- oth = skb_header_pointer(oldskb, ip_hdrlen(oldskb),
- sizeof(_otcph), &_otcph);
- if (oth == NULL)
- return;
-
- /* Check checksum. */
- if (nf_ip_checksum(oldskb, hook, ip_hdrlen(oldskb), IPPROTO_TCP))
- return;
-
- /*
- * Copy skb (even if skb is about to be dropped, we cannot just
- * clone it because there may be other things, such as tcpdump,
- * interested in it)
- */
- nskb = skb_copy_expand(oldskb, LL_MAX_HEADER,
- skb_tailroom(oldskb), GFP_ATOMIC);
- if (nskb == NULL)
- return;
-
- /* This packet will not be the same as the other: clear nf fields */
- nf_reset(nskb);
- skb_nfmark(nskb) = 0;
- skb_init_secmark(nskb);
-
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 18)
- skb_shinfo(nskb)->gso_size = 0;
- skb_shinfo(nskb)->gso_segs = 0;
- skb_shinfo(nskb)->gso_type = 0;
-#endif
-
- oldhdr = ip_hdr(oldskb);
- tcph = (struct tcphdr *)(skb_network_header(nskb) + ip_hdrlen(nskb));
-
- /* Swap source and dest */
- niph = ip_hdr(nskb);
- niph->daddr = xchg(&niph->saddr, niph->daddr);
- tmp = tcph->source;
- tcph->source = tcph->dest;
- tcph->dest = tmp;
-
- /* Calculate payload size?? */
- payload = nskb->len - ip_hdrlen(nskb) - sizeof(struct tcphdr);
-
- /* Truncate to length (no data) */
- tcph->doff = sizeof(struct tcphdr) / 4;
- skb_trim(nskb, ip_hdrlen(nskb) + sizeof(struct tcphdr));
- niph->tot_len = htons(nskb->len);
- tcph->urg_ptr = 0;
- /* Reset flags */
- ((u_int8_t *)tcph)[13] = 0;
+static void tarpit_generic(struct tcphdr *oth, struct tcphdr *tcph, uint16_t payload, unsigned int mode) {
if (mode == XTTARPIT_TARPIT) {
/* No replies for RST, FIN or !SYN,!ACK */
@@ -194,6 +131,74 @@ static void tarpit_tcp(struct sk_buff *oldskb, unsigned int hook,
tcph->seq = oth->ack_seq;
tcph->ack_seq = oth->seq;
}
+}
+
+static void tarpit_tcp(struct sk_buff *oldskb, unsigned int hook,
+ unsigned int mode)
+{
+ struct tcphdr _otcph, *oth, *tcph;
+ unsigned int addr_type = RTN_UNSPEC;
+ struct sk_buff *nskb;
+ const struct iphdr *oldhdr;
+ struct iphdr *niph;
+ uint16_t tmp, payload;
+
+ /* A truncated TCP header is not going to be useful */
+ if (oldskb->len < ip_hdrlen(oldskb) + sizeof(struct tcphdr))
+ return;
+
+ oth = skb_header_pointer(oldskb, ip_hdrlen(oldskb),
+ sizeof(_otcph), &_otcph);
+ if (oth == NULL)
+ return;
+
+ /* Check checksum. */
+ if (nf_ip_checksum(oldskb, hook, ip_hdrlen(oldskb), IPPROTO_TCP))
+ return;
+
+ /*
+ * Copy skb (even if skb is about to be dropped, we cannot just
+ * clone it because there may be other things, such as tcpdump,
+ * interested in it)
+ */
+ nskb = skb_copy_expand(oldskb, LL_MAX_HEADER,
+ skb_tailroom(oldskb), GFP_ATOMIC);
+ if (nskb == NULL)
+ return;
+
+ /* This packet will not be the same as the other: clear nf fields */
+ nf_reset(nskb);
+ skb_nfmark(nskb) = 0;
+ skb_init_secmark(nskb);
+
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 18)
+ skb_shinfo(nskb)->gso_size = 0;
+ skb_shinfo(nskb)->gso_segs = 0;
+ skb_shinfo(nskb)->gso_type = 0;
+#endif
+
+ oldhdr = ip_hdr(oldskb);
+ tcph = (struct tcphdr *)(skb_network_header(nskb) + ip_hdrlen(nskb));
+
+ /* Swap source and dest */
+ niph = ip_hdr(nskb);
+ niph->daddr = xchg(&niph->saddr, niph->daddr);
+ tmp = tcph->source;
+ tcph->source = tcph->dest;
+ tcph->dest = tmp;
+
+ /* Calculate payload size?? */
+ payload = nskb->len - ip_hdrlen(nskb) - sizeof(struct tcphdr);
+
+ /* Truncate to length (no data) */
+ tcph->doff = sizeof(struct tcphdr) / 4;
+ skb_trim(nskb, ip_hdrlen(nskb) + sizeof(struct tcphdr));
+ niph->tot_len = htons(nskb->len);
+ tcph->urg_ptr = 0;
+ /* Reset flags */
+ ((u_int8_t *)tcph)[13] = 0;
+
+ tarpit_generic(oth, tcph, payload, mode);
/* Adjust TCP checksum */
tcph->check = 0;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/3] xtables: tarpit: Add IPv6 support
2012-07-07 21:20 [PATCH 0/3] IPv6 tarpit support Josh Hunt
2012-07-07 21:20 ` [PATCH 1/3] xtables: tarpit: Make tarpit code generic Josh Hunt
@ 2012-07-07 21:21 ` Josh Hunt
2012-07-07 21:21 ` [PATCH 3/3] xtables: libxt_TARPIT: Enable " Josh Hunt
2 siblings, 0 replies; 6+ messages in thread
From: Josh Hunt @ 2012-07-07 21:21 UTC (permalink / raw)
To: netfilter-devel; +Cc: Josh Hunt
Adds support for an IPv6 tarpit target attempting to mimic the v4 implementation.
Signed-off-by: Josh Hunt <johunt@akamai.com>
---
extensions/xt_TARPIT.c | 215 ++++++++++++++++++++++++++++++++++++++++++++----
1 files changed, 200 insertions(+), 15 deletions(-)
diff --git a/extensions/xt_TARPIT.c b/extensions/xt_TARPIT.c
index 9a09e75..a84b9c8 100644
--- a/extensions/xt_TARPIT.c
+++ b/extensions/xt_TARPIT.c
@@ -51,7 +51,15 @@
#include "compat_xtables.h"
#include "xt_TARPIT.h"
-static void tarpit_generic(struct tcphdr *oth, struct tcphdr *tcph, uint16_t payload, unsigned int mode) {
+#include <net/ipv6.h>
+#include <linux/netfilter_ipv6.h>
+#include <net/ip6_route.h>
+#include <net/ip6_checksum.h>
+#include <net/addrconf.h>
+
+
+static void tarpit_generic(struct tcphdr *oth, struct tcphdr *tcph,
+ uint16_t payload, unsigned int mode) {
if (mode == XTTARPIT_TARPIT) {
/* No replies for RST, FIN or !SYN,!ACK */
@@ -133,7 +141,7 @@ static void tarpit_generic(struct tcphdr *oth, struct tcphdr *tcph, uint16_t pay
}
}
-static void tarpit_tcp(struct sk_buff *oldskb, unsigned int hook,
+static void tarpit_tcp4(struct sk_buff *oldskb, unsigned int hook,
unsigned int mode)
{
struct tcphdr _otcph, *oth, *tcph;
@@ -262,8 +270,130 @@ static void tarpit_tcp(struct sk_buff *oldskb, unsigned int hook,
kfree_skb(nskb);
}
+static void tarpit_tcp6(struct sk_buff *oldskb, unsigned int hook,
+ unsigned int mode)
+{
+ struct sk_buff *nskb;
+ struct tcphdr *tcph, oth;
+ unsigned int otcplen;
+ int tcphoff;
+ const struct ipv6hdr *oip6h = ipv6_hdr(oldskb);
+ struct ipv6hdr *ip6h;
+#define DEFAULT_TOS_VALUE 0x0U
+ const __u8 tclass = DEFAULT_TOS_VALUE;
+ u8 proto;
+ uint16_t payload;
+
+ proto = oip6h->nexthdr;
+ tcphoff = ipv6_skip_exthdr(oldskb, ((u8*)(oip6h+1) - oldskb->data), &proto);
+
+ if ((tcphoff < 0) || (tcphoff > oldskb->len)) {
+ pr_debug("Cannot get TCP header.\n");
+ return;
+ }
+
+ otcplen = oldskb->len - tcphoff;
+
+ /* IP header checks: fragment, too short. */
+ if (proto != IPPROTO_TCP || otcplen < sizeof(struct tcphdr)) {
+ pr_debug("proto(%d) != IPPROTO_TCP, "
+ "or too short. otcplen = %d\n",
+ proto, otcplen);
+ return;
+ }
+
+ if (skb_copy_bits(oldskb, tcphoff, &oth, sizeof(struct tcphdr)))
+ BUG();
+
+ /* Check checksum. */
+ if (csum_ipv6_magic(&oip6h->saddr, &oip6h->daddr, otcplen, IPPROTO_TCP,
+ skb_checksum(oldskb, tcphoff, otcplen, 0))) {
+ pr_debug("TCP checksum is invalid\n");
+ return;
+ }
+
+ nskb = skb_copy_expand(oldskb, LL_MAX_HEADER,
+ skb_tailroom(oldskb), GFP_ATOMIC);
+ if (!nskb) {
+ if (net_ratelimit())
+ pr_debug("cannot alloc skb\n");
+ return;
+ }
+
+ /* This packet will not be the same as the other: clear nf fields */
+ nf_reset(nskb);
+ skb_nfmark(nskb) = 0;
+ skb_init_secmark(nskb);
+
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 18)
+ skb_shinfo(nskb)->gso_size = 0;
+ skb_shinfo(nskb)->gso_segs = 0;
+ skb_shinfo(nskb)->gso_type = 0;
+#endif
+
+ skb_put(nskb, sizeof(struct ipv6hdr));
+ ip6h = ipv6_hdr(nskb);
+ *(__be32 *)ip6h = htonl(0x60000000 | (tclass << 20));
+ ip6h->nexthdr = IPPROTO_TCP;
+ ipv6_addr_copy(&ip6h->saddr, &oip6h->daddr);
+ ipv6_addr_copy(&ip6h->daddr, &oip6h->saddr);
+
+ /* Adjust IP TTL */
+ if (mode == XTTARPIT_HONEYPOT)
+ ip6h->hop_limit = 128;
+ else
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 26)
+ ip6h->hop_limit = ip6_dst_hoplimit(skb_dst(nskb));
+#else
+ ip6h->hop_limit = dst_metric(dst, RTAX_HOPLIMIT);
+ if (ip6h->hop_limit < 0)
+ ip6h->hop_limit = ipv6_get_hoplimit((skb_dst(nskb))->dev).
+#endif
+
+ tcph = (struct tcphdr *)(skb_network_header(nskb) + sizeof(struct ipv6hdr));
+
+ /* Truncate to length (no data) */
+ skb_trim(nskb, sizeof(struct ipv6hdr) + sizeof(struct tcphdr));
+ tcph->doff = sizeof(struct tcphdr)/4;
+ tcph->source = oth.dest;
+ tcph->dest = oth.source;
+
+ tcph->urg_ptr = 0;
+ /* Reset flags */
+ ((u_int8_t *)tcph)[13] = 0;
+
+ payload = nskb->len - sizeof(struct ipv6hdr) - sizeof(struct tcphdr);
+
+ tarpit_generic(&oth, tcph, payload, mode);
+
+ ip6h->payload_len = htons(sizeof(struct tcphdr));
+ tcph->check = 0;
+
+ /* Adjust TCP checksum */
+ tcph->check = csum_ipv6_magic(&ipv6_hdr(nskb)->saddr,
+ &ipv6_hdr(nskb)->daddr,
+ sizeof(struct tcphdr), IPPROTO_TCP,
+ csum_partial(tcph,
+ sizeof(struct tcphdr), 0));
+
+ if (ip6_route_me_harder(nskb))
+ goto free_nskb;
+
+ nskb->ip_summed = CHECKSUM_NONE;
+
+ nf_ct_attach(nskb, oldskb);
+
+ NF_HOOK(NFPROTO_IPV6, NF_INET_LOCAL_OUT, nskb, NULL,
+ skb_dst(nskb)->dev, dst_output);
+ return;
+
+free_nskb:
+ kfree_skb(nskb);
+
+}
+
static unsigned int
-tarpit_tg(struct sk_buff **pskb, const struct xt_action_param *par)
+tarpit_tg4(struct sk_buff **pskb, const struct xt_action_param *par)
{
const struct sk_buff *skb = *pskb;
const struct iphdr *iph = ip_hdr(skb);
@@ -294,29 +424,83 @@ tarpit_tg(struct sk_buff **pskb, const struct xt_action_param *par)
if (iph->frag_off & htons(IP_OFFSET))
return NF_DROP;
- tarpit_tcp(*pskb, par->hooknum, info->variant);
+ tarpit_tcp4(*pskb, par->hooknum, info->variant);
return NF_DROP;
}
-static struct xt_target tarpit_tg_reg __read_mostly = {
- .name = "TARPIT",
- .revision = 0,
- .family = NFPROTO_IPV4,
- .hooks = (1 << NF_INET_LOCAL_IN) | (1 << NF_INET_FORWARD),
- .proto = IPPROTO_TCP,
- .target = tarpit_tg,
- .targetsize = sizeof(struct xt_tarpit_tginfo),
- .me = THIS_MODULE,
+static unsigned int
+tarpit_tg6(struct sk_buff **pskb, const struct xt_action_param *par)
+{
+ const struct sk_buff *skb = *pskb;
+ const struct ipv6hdr *iph = ipv6_hdr(skb);
+ const struct rt6_info *rt = (struct rt6_info *)skb_dst(skb);
+ const struct xt_tarpit_tginfo *info = par->targinfo;
+ u8 proto;
+
+ /* Do we have an input route cache entry? (Not in PREROUTING.) */
+ if (rt == NULL) {
+ pr_debug("Dropping no input route cache entry\n");
+ return NF_DROP;
+ }
+
+ /* No replies to physical multicast/broadcast */
+ /* skb != PACKET_OTHERHOST handled by ip_rcv() */
+ if (skb->pkt_type != PACKET_HOST) {
+ pr_debug("type != PACKET_HOST");
+ return NF_DROP;
+ }
+
+ /*
+ * Our naive response construction does not deal with IP
+ * options, and probably should not try.
+ */
+ proto = iph->nexthdr;
+ if (ipv6_skip_exthdr(skb, skb_network_header_len(skb), &proto) != sizeof(struct ipv6hdr))
+ return NF_DROP;
+
+ if ((!(ipv6_addr_type(&iph->saddr) & IPV6_ADDR_UNICAST)) ||
+ (!(ipv6_addr_type(&iph->daddr) & IPV6_ADDR_UNICAST))) {
+ pr_debug("addr is not unicast.\n");
+ return;
+ }
+
+ tarpit_tcp6(*pskb, par->hooknum, info->variant);
+ return NF_DROP;
+}
+
+
+static struct xt_target tarpit_tg_reg[] __read_mostly = {
+ {
+ .name = "TARPIT",
+ .revision = 0,
+ .family = NFPROTO_IPV4,
+ .hooks = (1 << NF_INET_LOCAL_IN) | (1 << NF_INET_FORWARD),
+ .proto = IPPROTO_TCP,
+ .target = tarpit_tg4,
+ .targetsize = sizeof(struct xt_tarpit_tginfo),
+ .me = THIS_MODULE,
+ },
+ {
+ .name = "TARPIT",
+ .revision = 0,
+ .family = NFPROTO_IPV6,
+ .hooks = (1 << NF_INET_LOCAL_IN) | (1 << NF_INET_FORWARD),
+ .proto = IPPROTO_TCP,
+ .target = tarpit_tg6,
+ .targetsize = sizeof(struct xt_tarpit_tginfo),
+ .me = THIS_MODULE,
+ },
+
};
static int __init tarpit_tg_init(void)
{
- return xt_register_target(&tarpit_tg_reg);
+ return xt_register_targets(tarpit_tg_reg, ARRAY_SIZE(tarpit_tg_reg));
}
static void __exit tarpit_tg_exit(void)
{
- xt_unregister_target(&tarpit_tg_reg);
+ xt_unregister_targets(tarpit_tg_reg, ARRAY_SIZE(tarpit_tg_reg));
}
module_init(tarpit_tg_init);
@@ -325,3 +509,4 @@ MODULE_DESCRIPTION("Xtables: \"TARPIT\", capture and hold TCP connections");
MODULE_AUTHOR("Jan Engelhardt <jengelh@medozas.de>");
MODULE_LICENSE("GPL");
MODULE_ALIAS("ipt_TARPIT");
+MODULE_ALIAS("ip6t_TARPIT");
--
1.7.0.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/3] xtables: libxt_TARPIT: Enable IPv6 support
2012-07-07 21:20 [PATCH 0/3] IPv6 tarpit support Josh Hunt
2012-07-07 21:20 ` [PATCH 1/3] xtables: tarpit: Make tarpit code generic Josh Hunt
2012-07-07 21:21 ` [PATCH 2/3] xtables: tarpit: Add IPv6 support Josh Hunt
@ 2012-07-07 21:21 ` Josh Hunt
2 siblings, 0 replies; 6+ messages in thread
From: Josh Hunt @ 2012-07-07 21:21 UTC (permalink / raw)
To: netfilter-devel; +Cc: Josh Hunt
Enables userspace IPv6 tarpit support.
Signed-off-by: Josh Hunt <johunt@akamai.com>
---
extensions/libxt_TARPIT.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/extensions/libxt_TARPIT.c b/extensions/libxt_TARPIT.c
index 59c190f..6b65b09 100644
--- a/extensions/libxt_TARPIT.c
+++ b/extensions/libxt_TARPIT.c
@@ -106,7 +106,7 @@ static void tarpit_tg_save(const void *ip,
static struct xtables_target tarpit_tg_reg = {
.version = XTABLES_VERSION,
.name = "TARPIT",
- .family = NFPROTO_IPV4,
+ .family = NFPROTO_UNSPEC,
.size = XT_ALIGN(sizeof(struct xt_tarpit_tginfo)),
.userspacesize = XT_ALIGN(sizeof(struct xt_tarpit_tginfo)),
.help = tarpit_tg_help,
--
1.7.0.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] xtables: tarpit: Make tarpit code generic
2012-07-07 21:20 ` [PATCH 1/3] xtables: tarpit: Make tarpit code generic Josh Hunt
@ 2012-07-08 12:58 ` Jan Engelhardt
2012-07-08 13:58 ` Josh Hunt
0 siblings, 1 reply; 6+ messages in thread
From: Jan Engelhardt @ 2012-07-08 12:58 UTC (permalink / raw)
To: Josh Hunt; +Cc: netfilter-devel
On Saturday 2012-07-07 23:20, Josh Hunt wrote:
>Moves TCP header manipulation into a separate function to allow code-sharing for
>IPv6 support.
Hi Josh,
in your patch, you seem to oversee that the tarpit_generic block that
you are moving into a new function had "return;"s to exit from
the tarpit_tcp function. With your change,
>+ niph->tot_len = htons(nskb->len);
>+ tcph->urg_ptr = 0;
>+ /* Reset flags */
>+ ((u_int8_t *)tcph)[13] = 0;
>+
>+ tarpit_generic(oth, tcph, payload, mode);
>
> /* Adjust TCP checksum */
> tcph->check = 0;
The checksum code (and what follows) it will be executed in all cases,
even though TCP RST are supposed not to get any reply per the comment.
tarpit_generic would have to return a bool value propagating this
exit style. In other words,
static bool tarpit_generic
{
if (mode == XTTARPIT_TARPIT)
/* No replies for RST, FIN , ... */
if (oth->rst ...)
return false;
}
...
return true;
}
static void tarpit_tcp
{
...
if (!tarpit_generic())
return;
tcph->check = 0;
...
}
It would also be appreciated if you could move each case
(XTTARPIT_{TARPIT,HONEYPOT,RESET}) into their own functions in three
patches added to the front of the set.
(Reduces indent and makes the diffs simpler.)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] xtables: tarpit: Make tarpit code generic
2012-07-08 12:58 ` Jan Engelhardt
@ 2012-07-08 13:58 ` Josh Hunt
0 siblings, 0 replies; 6+ messages in thread
From: Josh Hunt @ 2012-07-08 13:58 UTC (permalink / raw)
To: Jan Engelhardt; +Cc: netfilter-devel
On 07/08/2012 07:58 AM, Jan Engelhardt wrote:
>
> On Saturday 2012-07-07 23:20, Josh Hunt wrote:
>
>> Moves TCP header manipulation into a separate function to allow code-sharing for
>> IPv6 support.
>
> Hi Josh,
>
>
> in your patch, you seem to oversee that the tarpit_generic block that
> you are moving into a new function had "return;"s to exit from
> the tarpit_tcp function. With your change,
>
>> + niph->tot_len = htons(nskb->len);
>> + tcph->urg_ptr = 0;
>> + /* Reset flags */
>> + ((u_int8_t *)tcph)[13] = 0;
>> +
>> + tarpit_generic(oth, tcph, payload, mode);
>>
>> /* Adjust TCP checksum */
>> tcph->check = 0;
>
> The checksum code (and what follows) it will be executed in all cases,
> even though TCP RST are supposed not to get any reply per the comment.
>
> tarpit_generic would have to return a bool value propagating this
> exit style. In other words,
>
>
> static bool tarpit_generic
> {
> if (mode == XTTARPIT_TARPIT)
> /* No replies for RST, FIN , ... */
> if (oth->rst ...)
> return false;
> }
> ...
> return true;
> }
>
> static void tarpit_tcp
> {
> ...
> if (!tarpit_generic())
> return;
> tcph->check = 0;
> ...
> }
Doh! Yep totally overlooked that. Thanks for catching. Will update and
resend.
>
> It would also be appreciated if you could move each case
> (XTTARPIT_{TARPIT,HONEYPOT,RESET}) into their own functions in three
> patches added to the front of the set.
> (Reduces indent and makes the diffs simpler.)
>
Sure. I'll do this as well.
Thanks for the review.
Josh
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-07-08 13:58 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-07 21:20 [PATCH 0/3] IPv6 tarpit support Josh Hunt
2012-07-07 21:20 ` [PATCH 1/3] xtables: tarpit: Make tarpit code generic Josh Hunt
2012-07-08 12:58 ` Jan Engelhardt
2012-07-08 13:58 ` Josh Hunt
2012-07-07 21:21 ` [PATCH 2/3] xtables: tarpit: Add IPv6 support Josh Hunt
2012-07-07 21:21 ` [PATCH 3/3] xtables: libxt_TARPIT: Enable " Josh Hunt
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.