WARNING: line length of 84 exceeds 80 columns #59: FILE: net/ipv4/icmp.c:812: + if ( !(ct->status & IPS_SRC_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) ERROR: space prohibited after that open parenthesis '(' #59: FILE: net/ipv4/icmp.c:812: + if ( !(ct->status & IPS_SRC_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) WARNING: line length of 90 exceeds 80 columns #60: FILE: net/ipv4/icmp.c:813: + && !(ct->status & IPS_DST_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_REPLY)) { CHECK: Logical continuations should be on the previous line #60: FILE: net/ipv4/icmp.c:813: + if ( !(ct->status & IPS_SRC_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) + && !(ct->status & IPS_DST_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_REPLY)) { WARNING: braces {} are not necessary for any arm of this statement #69: FILE: net/ipv4/icmp.c:828: + if (CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) { [...] + } else { [...] WARNING: line length of 90 exceeds 80 columns #70: FILE: net/ipv4/icmp.c:829: + ip_hdr(skb_in)->saddr = ct->tuplehash[IP_CT_DIR_ORIGINAL].tuple.src.u3.ip; WARNING: line length of 87 exceeds 80 columns #72: FILE: net/ipv4/icmp.c:831: + ip_hdr(skb_in)->saddr = ct->tuplehash[IP_CT_DIR_REPLY].tuple.src.u3.ip; WARNING: line length of 84 exceeds 80 columns #91: FILE: net/ipv6/ip6_icmp.c:66: + if ( !(ct->status & IPS_SRC_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) ERROR: space prohibited after that open parenthesis '(' #91: FILE: net/ipv6/ip6_icmp.c:66: + if ( !(ct->status & IPS_SRC_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) WARNING: line length of 90 exceeds 80 columns #92: FILE: net/ipv6/ip6_icmp.c:67: + && !(ct->status & IPS_DST_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_REPLY)) { CHECK: Logical continuations should be on the previous line #92: FILE: net/ipv6/ip6_icmp.c:67: + if ( !(ct->status & IPS_SRC_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) + && !(ct->status & IPS_DST_NAT && CTINFO2DIR(ctinfo) == IP_CT_DIR_REPLY)) { WARNING: braces {} are not necessary for any arm of this statement #101: FILE: net/ipv6/ip6_icmp.c:82: + if (CTINFO2DIR(ctinfo) == IP_CT_DIR_ORIGINAL) { [...] + } else { [...] WARNING: line length of 93 exceeds 80 columns #102: FILE: net/ipv6/ip6_icmp.c:83: + ipv6_hdr(skb_in)->saddr = ct->tuplehash[IP_CT_DIR_ORIGINAL].tuple.src.u3.in6; WARNING: line length of 90 exceeds 80 columns #104: FILE: net/ipv6/ip6_icmp.c:85: + ipv6_hdr(skb_in)->saddr = ct->tuplehash[IP_CT_DIR_REPLY].tuple.src.u3.in6; total: 2 errors, 10 warnings, 2 checks, 52 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit dffa13d42380 ("icmp: fix icmp_ndo_send address translation for reply direction") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.