Commit 91e8db80 authored by Patrick McHardy's avatar Patrick McHardy Committed by David S. Miller

[NETFILTER]: nf_conntrack_h323: turn some printks into DEBUGPs

Don't spam the ringbuffer with decoding errors. The only printks remaining
are for dropped packets when we're certain they are H.323.
Signed-off-by: default avatarPatrick McHardy <kaber@trash.net>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent d3c3f424
...@@ -163,9 +163,7 @@ static int get_tpkt_data(struct sk_buff **pskb, unsigned int protoff, ...@@ -163,9 +163,7 @@ static int get_tpkt_data(struct sk_buff **pskb, unsigned int protoff,
} }
/* Fragmented TPKT */ /* Fragmented TPKT */
if (net_ratelimit()) DEBUGP("nf_ct_h323: fragmented TPKT\n");
printk("nf_ct_h323: "
"fragmented TPKT\n");
goto clear_out; goto clear_out;
} }
...@@ -596,10 +594,9 @@ static int h245_help(struct sk_buff **pskb, unsigned int protoff, ...@@ -596,10 +594,9 @@ static int h245_help(struct sk_buff **pskb, unsigned int protoff,
ret = DecodeMultimediaSystemControlMessage(data, datalen, ret = DecodeMultimediaSystemControlMessage(data, datalen,
&mscm); &mscm);
if (ret < 0) { if (ret < 0) {
if (net_ratelimit()) DEBUGP("nf_ct_h245: decoding error: %s\n",
printk("nf_ct_h245: decoding error: %s\n", ret == H323_ERROR_BOUND ?
ret == H323_ERROR_BOUND ? "out of bound" : "out of range");
"out of bound" : "out of range");
/* We don't drop when decoding error */ /* We don't drop when decoding error */
break; break;
} }
...@@ -1137,10 +1134,9 @@ static int q931_help(struct sk_buff **pskb, unsigned int protoff, ...@@ -1137,10 +1134,9 @@ static int q931_help(struct sk_buff **pskb, unsigned int protoff,
/* Decode Q.931 signal */ /* Decode Q.931 signal */
ret = DecodeQ931(data, datalen, &q931); ret = DecodeQ931(data, datalen, &q931);
if (ret < 0) { if (ret < 0) {
if (net_ratelimit()) DEBUGP("nf_ct_q931: decoding error: %s\n",
printk("nf_ct_q931: decoding error: %s\n", ret == H323_ERROR_BOUND ?
ret == H323_ERROR_BOUND ? "out of bound" : "out of range");
"out of bound" : "out of range");
/* We don't drop when decoding error */ /* We don't drop when decoding error */
break; break;
} }
...@@ -1711,10 +1707,9 @@ static int ras_help(struct sk_buff **pskb, unsigned int protoff, ...@@ -1711,10 +1707,9 @@ static int ras_help(struct sk_buff **pskb, unsigned int protoff,
/* Decode RAS message */ /* Decode RAS message */
ret = DecodeRasMessage(data, datalen, &ras); ret = DecodeRasMessage(data, datalen, &ras);
if (ret < 0) { if (ret < 0) {
if (net_ratelimit()) DEBUGP("nf_ct_ras: decoding error: %s\n",
printk("nf_ct_ras: decoding error: %s\n", ret == H323_ERROR_BOUND ?
ret == H323_ERROR_BOUND ? "out of bound" : "out of range");
"out of bound" : "out of range");
goto accept; goto accept;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment