Skip to content

Commit

Permalink
netfilter 01/09: remove "happy cracking" message
Browse files Browse the repository at this point in the history
Don't spam logs for locally generated short packets. these can only
be generated by root.

Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Patrick McHardy authored and David S. Miller committed Jan 13, 2009
1 parent 985ebdb commit 8884310
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 25 deletions.
7 changes: 1 addition & 6 deletions net/ipv4/netfilter/iptable_filter.c
Original file line number Diff line number Diff line change
Expand Up @@ -93,13 +93,8 @@ ipt_local_out_hook(unsigned int hook,
{
/* root is playing with raw sockets. */
if (skb->len < sizeof(struct iphdr) ||
ip_hdrlen(skb) < sizeof(struct iphdr)) {
if (net_ratelimit())
printk("iptable_filter: ignoring short SOCK_RAW "
"packet.\n");
ip_hdrlen(skb) < sizeof(struct iphdr))
return NF_ACCEPT;
}

return ipt_do_table(skb, hook, in, out,
dev_net(out)->ipv4.iptable_filter);
}
Expand Down
6 changes: 1 addition & 5 deletions net/ipv4/netfilter/iptable_mangle.c
Original file line number Diff line number Diff line change
Expand Up @@ -132,12 +132,8 @@ ipt_local_hook(unsigned int hook,

/* root is playing with raw sockets. */
if (skb->len < sizeof(struct iphdr)
|| ip_hdrlen(skb) < sizeof(struct iphdr)) {
if (net_ratelimit())
printk("iptable_mangle: ignoring short SOCK_RAW "
"packet.\n");
|| ip_hdrlen(skb) < sizeof(struct iphdr))
return NF_ACCEPT;
}

/* Save things which could affect route */
mark = skb->mark;
Expand Down
6 changes: 1 addition & 5 deletions net/ipv4/netfilter/iptable_raw.c
Original file line number Diff line number Diff line change
Expand Up @@ -65,12 +65,8 @@ ipt_local_hook(unsigned int hook,
{
/* root is playing with raw sockets. */
if (skb->len < sizeof(struct iphdr) ||
ip_hdrlen(skb) < sizeof(struct iphdr)) {
if (net_ratelimit())
printk("iptable_raw: ignoring short SOCK_RAW "
"packet.\n");
ip_hdrlen(skb) < sizeof(struct iphdr))
return NF_ACCEPT;
}
return ipt_do_table(skb, hook, in, out,
dev_net(out)->ipv4.iptable_raw);
}
Expand Down
6 changes: 1 addition & 5 deletions net/ipv4/netfilter/iptable_security.c
Original file line number Diff line number Diff line change
Expand Up @@ -96,12 +96,8 @@ ipt_local_out_hook(unsigned int hook,
{
/* Somebody is playing with raw sockets. */
if (skb->len < sizeof(struct iphdr)
|| ip_hdrlen(skb) < sizeof(struct iphdr)) {
if (net_ratelimit())
printk(KERN_INFO "iptable_security: ignoring short "
"SOCK_RAW packet.\n");
|| ip_hdrlen(skb) < sizeof(struct iphdr))
return NF_ACCEPT;
}
return ipt_do_table(skb, hook, in, out,
dev_net(out)->ipv4.iptable_security);
}
Expand Down
5 changes: 1 addition & 4 deletions net/ipv4/netfilter/nf_conntrack_l3proto_ipv4.c
Original file line number Diff line number Diff line change
Expand Up @@ -145,11 +145,8 @@ static unsigned int ipv4_conntrack_local(unsigned int hooknum,
{
/* root is playing with raw sockets. */
if (skb->len < sizeof(struct iphdr) ||
ip_hdrlen(skb) < sizeof(struct iphdr)) {
if (net_ratelimit())
printk("ipt_hook: happy cracking.\n");
ip_hdrlen(skb) < sizeof(struct iphdr))
return NF_ACCEPT;
}
return nf_conntrack_in(dev_net(out), PF_INET, hooknum, skb);
}

Expand Down

0 comments on commit 8884310

Please sign in to comment.