diff options
author | Patrick McHardy <kaber@trash.net> | 2010-06-28 08:12:41 -0400 |
---|---|---|
committer | Patrick McHardy <kaber@trash.net> | 2010-06-28 08:12:41 -0400 |
commit | cf377eb4aeded926375d4d0fe0b66ba95f0521e1 (patch) | |
tree | 010628b36c81d050f4189fb8e94f0cdec60dc3c0 /net/ipv4 | |
parent | d70a011dbbaa6335a19deb63ec3eb613f48faafd (diff) |
netfilter: ipt_LOG/ip6t_LOG: remove comparison within loop
Remove the comparison within the loop to print the macheader by prepending
the colon to all but the first printk.
Based on suggestion by Jan Engelhardt <jengelh@medozas.de>.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Diffstat (limited to 'net/ipv4')
-rw-r--r-- | net/ipv4/netfilter/ipt_LOG.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/net/ipv4/netfilter/ipt_LOG.c b/net/ipv4/netfilter/ipt_LOG.c index 5234f4f3499a..0a452a54adbe 100644 --- a/net/ipv4/netfilter/ipt_LOG.c +++ b/net/ipv4/netfilter/ipt_LOG.c | |||
@@ -411,12 +411,12 @@ ipt_log_packet(u_int8_t pf, | |||
411 | skb->mac_header != skb->network_header) { | 411 | skb->mac_header != skb->network_header) { |
412 | int i; | 412 | int i; |
413 | const unsigned char *p = skb_mac_header(skb); | 413 | const unsigned char *p = skb_mac_header(skb); |
414 | for (i = 0; i < skb->dev->hard_header_len; i++,p++) | 414 | |
415 | printk("%02x%c", *p, | 415 | printk("%02x", *p++); |
416 | i==skb->dev->hard_header_len - 1 | 416 | for (i = 1; i < skb->dev->hard_header_len; i++, p++) |
417 | ? ' ':':'); | 417 | printk(":%02x", *p); |
418 | } else | 418 | } |
419 | printk(" "); | 419 | printk(" "); |
420 | } | 420 | } |
421 | 421 | ||
422 | dump_packet(loginfo, skb, 0); | 422 | dump_packet(loginfo, skb, 0); |