git: c730d0dcc309 - stable/12 - ipfilter module: Style(9) requires a space after return
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Mon, 07 Feb 2022 13:56:04 UTC
The branch stable/12 has been updated by cy:
URL: https://cgit.FreeBSD.org/src/commit/?id=c730d0dcc30972c420219738809be28bf972fe88
commit c730d0dcc30972c420219738809be28bf972fe88
Author: Cy Schubert <cy@FreeBSD.org>
AuthorDate: 2022-01-04 02:42:49 +0000
Commit: Cy Schubert <cy@FreeBSD.org>
CommitDate: 2022-02-07 13:55:13 +0000
ipfilter module: Style(9) requires a space after return
Reported by: jrtc27
Fixes: 8c82b37461fa4e60276639df214100fbf559ea6e
(cherry picked from commit 701301511f0ca044fc71470ed7842cd2c6fee001)
---
sys/netpfil/ipfilter/netinet/fil.c | 664 +++++++++++++-------------
sys/netpfil/ipfilter/netinet/ip_auth.c | 70 +--
sys/netpfil/ipfilter/netinet/ip_dns_pxy.c | 44 +-
sys/netpfil/ipfilter/netinet/ip_dstlist.c | 92 ++--
sys/netpfil/ipfilter/netinet/ip_fil_freebsd.c | 134 +++---
sys/netpfil/ipfilter/netinet/ip_frag.c | 84 ++--
sys/netpfil/ipfilter/netinet/ip_ftp_pxy.c | 190 ++++----
sys/netpfil/ipfilter/netinet/ip_htable.c | 120 ++---
sys/netpfil/ipfilter/netinet/ip_ipsec_pxy.c | 40 +-
sys/netpfil/ipfilter/netinet/ip_irc_pxy.c | 64 +--
sys/netpfil/ipfilter/netinet/ip_log.c | 64 +--
sys/netpfil/ipfilter/netinet/ip_lookup.c | 66 +--
sys/netpfil/ipfilter/netinet/ip_nat.c | 388 +++++++--------
sys/netpfil/ipfilter/netinet/ip_nat6.c | 190 ++++----
sys/netpfil/ipfilter/netinet/ip_netbios_pxy.c | 4 +-
sys/netpfil/ipfilter/netinet/ip_pool.c | 118 ++---
sys/netpfil/ipfilter/netinet/ip_pptp_pxy.c | 26 +-
sys/netpfil/ipfilter/netinet/ip_proxy.c | 96 ++--
sys/netpfil/ipfilter/netinet/ip_raudio_pxy.c | 28 +-
sys/netpfil/ipfilter/netinet/ip_rcmd_pxy.c | 30 +-
sys/netpfil/ipfilter/netinet/ip_rpcb_pxy.c | 176 +++----
sys/netpfil/ipfilter/netinet/ip_rules.c | 28 +-
sys/netpfil/ipfilter/netinet/ip_scan.c | 58 +--
sys/netpfil/ipfilter/netinet/ip_state.c | 258 +++++-----
sys/netpfil/ipfilter/netinet/ip_sync.c | 60 +--
sys/netpfil/ipfilter/netinet/ip_tftp_pxy.c | 36 +-
sys/netpfil/ipfilter/netinet/ipf_rb.h | 6 +-
sys/netpfil/ipfilter/netinet/mlfk_ipl.c | 42 +-
sys/netpfil/ipfilter/netinet/radix_ipf.c | 60 +--
29 files changed, 1618 insertions(+), 1618 deletions(-)
diff --git a/sys/netpfil/ipfilter/netinet/fil.c b/sys/netpfil/ipfilter/netinet/fil.c
index ca0c12a2dddf..083b4f58b10f 100644
--- a/sys/netpfil/ipfilter/netinet/fil.c
+++ b/sys/netpfil/ipfilter/netinet/fil.c
@@ -624,12 +624,12 @@ ipf_pr_ipv6exthdr(fr_info_t *fin, int multiple, int proto)
if ((fin->fin_dlen - 8) < 0) {
fin->fin_flx |= FI_SHORT;
LBUMPD(ipf_stats[fin->fin_out], fr_v6_ext_short);
- return(NULL);
+ return (NULL);
}
if (ipf_pr_pullup(fin, 8) == -1) {
LBUMPD(ipf_stats[fin->fin_out], fr_v6_ext_pullup);
- return(NULL);
+ return (NULL);
}
hdr = fin->fin_dp;
@@ -647,7 +647,7 @@ ipf_pr_ipv6exthdr(fr_info_t *fin, int multiple, int proto)
fin->fin_flx |= FI_BAD;
DT3(ipf_fi_bad_pr_ipv6exthdr_len, fr_info_t *, fin, u_short, shift, u_short, fin->fin_dlen);
LBUMPD(ipf_stats[fin->fin_out], fr_v6_ext_hlen);
- return(NULL);
+ return (NULL);
}
fin->fin_dp = (char *)fin->fin_dp + shift;
@@ -659,7 +659,7 @@ ipf_pr_ipv6exthdr(fr_info_t *fin, int multiple, int proto)
* end result until after it has been defragmented.
*/
if (fin->fin_flx & FI_FRAG)
- return(hdr);
+ return (hdr);
for (i = 0; ip6exthdr[i].ol_bit != 0; i++)
if (ip6exthdr[i].ol_val == proto) {
@@ -675,7 +675,7 @@ ipf_pr_ipv6exthdr(fr_info_t *fin, int multiple, int proto)
break;
}
- return(hdr);
+ return (hdr);
}
@@ -694,8 +694,8 @@ ipf_pr_hopopts6(fr_info_t *fin)
hdr = ipf_pr_ipv6exthdr(fin, 0, IPPROTO_HOPOPTS);
if (hdr == NULL)
- return(IPPROTO_NONE);
- return(hdr->ip6e_nxt);
+ return (IPPROTO_NONE);
+ return (hdr->ip6e_nxt);
}
@@ -714,8 +714,8 @@ ipf_pr_mobility6(fr_info_t *fin)
hdr = ipf_pr_ipv6exthdr(fin, 0, IPPROTO_MOBILITY);
if (hdr == NULL)
- return(IPPROTO_NONE);
- return(hdr->ip6e_nxt);
+ return (IPPROTO_NONE);
+ return (hdr->ip6e_nxt);
}
@@ -734,7 +734,7 @@ ipf_pr_routing6(fr_info_t *fin)
hdr = (struct ip6_routing *)ipf_pr_ipv6exthdr(fin, 0, IPPROTO_ROUTING);
if (hdr == NULL)
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
switch (hdr->ip6r_type)
{
@@ -749,7 +749,7 @@ ipf_pr_routing6(fr_info_t *fin)
fin->fin_flx |= FI_BAD;
DT1(ipf_fi_bad_routing6, fr_info_t *, fin);
LBUMPD(ipf_stats[fin->fin_out], fr_v6_rh_bad);
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
}
break;
@@ -757,7 +757,7 @@ ipf_pr_routing6(fr_info_t *fin)
break;
}
- return(hdr->ip6r_nxt);
+ return (hdr->ip6r_nxt);
}
@@ -801,7 +801,7 @@ ipf_pr_fragment6(fr_info_t *fin)
frag = (struct ip6_frag *)ipf_pr_ipv6exthdr(fin, 0, IPPROTO_FRAGMENT);
if (frag == NULL) {
LBUMPD(ipf_stats[fin->fin_out], fr_v6_frag_bad);
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
}
if ((frag->ip6f_offlg & IP6F_MORE_FRAG) != 0) {
@@ -836,7 +836,7 @@ ipf_pr_fragment6(fr_info_t *fin)
* this effectively disables the fragment cache for > 1 protocol at a
* time.
*/
- return(frag->ip6f_nxt);
+ return (frag->ip6f_nxt);
}
@@ -857,9 +857,9 @@ ipf_pr_dstopts6(fr_info_t *fin)
hdr = ipf_pr_ipv6exthdr(fin, 0, IPPROTO_DSTOPTS);
if (hdr == NULL) {
LBUMPD(ipf_stats[fin->fin_out], fr_v6_dst_bad);
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
}
- return(hdr->ip6e_nxt);
+ return (hdr->ip6e_nxt);
}
@@ -1043,7 +1043,7 @@ ipf_pr_ah6(fr_info_t *fin)
ipf_main_softc_t *softc = fin->fin_main_soft;
LBUMPD(ipf_stats[fin->fin_out], fr_v6_ah_bad);
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
}
ipf_pr_short6(fin, sizeof(*ah));
@@ -1052,7 +1052,7 @@ ipf_pr_ah6(fr_info_t *fin)
* No need for another pullup, ipf_pr_ipv6exthdr() will pullup
* enough data to satisfy ah_next (the very first one.)
*/
- return(ah->ah_next);
+ return (ah->ah_next);
}
@@ -1115,7 +1115,7 @@ ipf_pr_pullup(fr_info_t *fin, int plen)
if (ipf_pullup(fin->fin_m, fin, plen) == NULL) {
DT(ipf_pullup_fail);
LBUMP(ipf_stats[fin->fin_out].fr_pull[1]);
- return(-1);
+ return (-1);
}
LBUMP(ipf_stats[fin->fin_out].fr_pull[0]);
#else
@@ -1127,11 +1127,11 @@ ipf_pr_pullup(fr_info_t *fin, int plen)
*fin->fin_mp = NULL;
fin->fin_m = NULL;
fin->fin_ip = NULL;
- return(-1);
+ return (-1);
#endif
}
}
- return(0);
+ return (0);
}
@@ -1302,12 +1302,12 @@ ipf_pr_tcpcommon(fr_info_t *fin)
fin->fin_flx |= FI_TCPUDP;
if (fin->fin_off != 0) {
LBUMPD(ipf_stats[fin->fin_out], fr_tcp_frag);
- return(0);
+ return (0);
}
if (ipf_pr_pullup(fin, sizeof(*tcp)) == -1) {
LBUMPD(ipf_stats[fin->fin_out], fr_tcp_pullup);
- return(-1);
+ return (-1);
}
tcp = fin->fin_dp;
@@ -1318,7 +1318,7 @@ ipf_pr_tcpcommon(fr_info_t *fin)
if ((fin->fin_flx & FI_SHORT) != 0) {
LBUMPD(ipf_stats[fin->fin_out], fr_tcp_short);
- return(1);
+ return (1);
}
/*
@@ -1330,7 +1330,7 @@ ipf_pr_tcpcommon(fr_info_t *fin)
LBUMPD(ipf_stats[fin->fin_out], fr_tcp_small);
fin->fin_flx |= FI_BAD;
DT3(ipf_fi_bad_tlen, fr_info_t, fin, u_int, tlen, u_int, sizeof(tcphdr_t));
- return(1);
+ return (1);
}
flags = tcp->th_flags;
@@ -1396,7 +1396,7 @@ ipf_pr_tcpcommon(fr_info_t *fin)
}
if (fin->fin_flx & FI_BAD) {
LBUMPD(ipf_stats[fin->fin_out], fr_tcp_bad_flags);
- return(1);
+ return (1);
}
/*
@@ -1408,12 +1408,12 @@ ipf_pr_tcpcommon(fr_info_t *fin)
* then that might add some weight to adding this...
*/
if (tlen == sizeof(tcphdr_t)) {
- return(0);
+ return (0);
}
if (ipf_pr_pullup(fin, tlen) == -1) {
LBUMPD(ipf_stats[fin->fin_out], fr_tcp_pullup);
- return(-1);
+ return (-1);
}
#if 0
@@ -1455,7 +1455,7 @@ ipf_pr_tcpcommon(fr_info_t *fin)
}
#endif /* 0 */
- return(0);
+ return (0);
}
@@ -1481,7 +1481,7 @@ ipf_pr_udpcommon(fr_info_t *fin)
fin->fin_flx |= FI_SHORT;
LBUMPD(ipf_stats[fin->fin_out], fr_udp_pullup);
- return(1);
+ return (1);
}
udp = fin->fin_dp;
@@ -1490,7 +1490,7 @@ ipf_pr_udpcommon(fr_info_t *fin)
fin->fin_dport = ntohs(udp->uh_dport);
}
- return(0);
+ return (0);
}
@@ -1579,13 +1579,13 @@ ipf_pr_ah(fr_info_t *fin)
if (((fin->fin_flx & FI_SHORT) != 0) || (fin->fin_off != 0)) {
LBUMPD(ipf_stats[fin->fin_out], fr_v4_ah_bad);
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
}
if (ipf_pr_pullup(fin, sizeof(*ah)) == -1) {
DT(fr_v4_ah_pullup_1);
LBUMP(ipf_stats[fin->fin_out].fr_v4_ah_pullup);
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
}
ah = (authhdr_t *)fin->fin_dp;
@@ -1595,7 +1595,7 @@ ipf_pr_ah(fr_info_t *fin)
if (ipf_pr_pullup(fin, len) == -1) {
DT(fr_v4_ah_pullup_2);
LBUMP(ipf_stats[fin->fin_out].fr_v4_ah_pullup);
- return(IPPROTO_NONE);
+ return (IPPROTO_NONE);
}
/*
@@ -1604,7 +1604,7 @@ ipf_pr_ah(fr_info_t *fin)
*/
fin->fin_dp = (char *)fin->fin_dp + len;
fin->fin_dlen -= len;
- return(ah->ah_next);
+ return (ah->ah_next);
}
@@ -1879,7 +1879,7 @@ ipf_checkripso(u_char *s)
m--;
}
- return(secmsk << 16) | auth;
+ return (secmsk << 16) | auth;
}
@@ -1911,7 +1911,7 @@ ipf_checkcipso(fr_info_t *fin, u_char *s, int ol)
LBUMPD(ipf_stats[fin->fin_out], fr_v4_cipso_bad);
fin->fin_flx |= FI_BAD;
DT2(ipf_fi_bad_checkcipso_ol, fr_info_t *, fin, u_int, ol);
- return(0);
+ return (0);
}
fi = &fin->fin_fi;
@@ -1929,7 +1929,7 @@ ipf_checkcipso(fr_info_t *fin, u_char *s, int ol)
LBUMPD(ipf_stats[fin->fin_out], fr_v4_cipso_tlen);
fin->fin_flx |= FI_BAD;
DT2(ipf_fi_bad_checkcipso_tlen, fr_info_t *, fin, u_int, tlen);
- return(0);
+ return (0);
}
sensitivity = 0;
@@ -1981,7 +1981,7 @@ ipf_checkcipso(fr_info_t *fin, u_char *s, int ol)
fi->fi_sensitivity = sensitivity;
}
- return(doi);
+ return (doi);
}
@@ -2027,9 +2027,9 @@ ipf_makefrip(int hlen, ip_t *ip, fr_info_t *fin)
}
if (fin->fin_ip == NULL) {
LBUMP(ipf_stats[fin->fin_out].fr_ip_freed);
- return(-1);
+ return (-1);
}
- return(0);
+ return (0);
}
@@ -2094,7 +2094,7 @@ ipf_portcheck(frpcmp_t *frp, u_32_t pop)
default :
break;
}
- return(err);
+ return (err);
}
@@ -2135,7 +2135,7 @@ ipf_tcpudpchk(fr_ip_t *fi, frtuc_t *ft)
*/
if (err && (fi->fi_p == IPPROTO_TCP)) {
if (fi->fi_flx & FI_SHORT)
- return(!(ft->ftu_tcpf | ft->ftu_tcpfm));
+ return (!(ft->ftu_tcpf | ft->ftu_tcpfm));
/*
* Match the flags ? If not, abort this match.
*/
@@ -2146,7 +2146,7 @@ ipf_tcpudpchk(fr_ip_t *fi, frtuc_t *ft)
err = 0;
}
}
- return(err);
+ return (err);
}
@@ -2184,7 +2184,7 @@ ipf_check_ipf(fr_info_t *fin, frentry_t *fr, int portcmp)
FR_DEBUG(("0. %#08x & %#08x != %#08x\n",
ntohl(*lip), ntohl(*lm), ntohl(*ld)));
if (i)
- return(1);
+ return (1);
/*
* Next 32 bits is a constructed bitmask indicating which IP options
@@ -2195,7 +2195,7 @@ ipf_check_ipf(fr_info_t *fin, frentry_t *fr, int portcmp)
FR_DEBUG(("1. %#08x & %#08x != %#08x\n",
ntohl(*lip), ntohl(*lm), ntohl(*ld)));
if (i != 0)
- return(1);
+ return (1);
lip++, lm++, ld++;
/*
@@ -2208,7 +2208,7 @@ ipf_check_ipf(fr_info_t *fin, frentry_t *fr, int portcmp)
i = (*fr->fr_srcfunc)(fin->fin_main_soft, fr->fr_srcptr,
fi->fi_v, lip, fin->fin_plen);
if (i == -1)
- return(1);
+ return (1);
lip += 3;
lm += 3;
ld += 3;
@@ -2237,7 +2237,7 @@ ipf_check_ipf(fr_info_t *fin, frentry_t *fr, int portcmp)
}
i ^= (fr->fr_flags & FR_NOTSRCIP) >> 6;
if (i != 0)
- return(1);
+ return (1);
/*
* Check the destination address.
@@ -2247,7 +2247,7 @@ ipf_check_ipf(fr_info_t *fin, frentry_t *fr, int portcmp)
i = (*fr->fr_dstfunc)(fin->fin_main_soft, fr->fr_dstptr,
fi->fi_v, lip, fin->fin_plen);
if (i == -1)
- return(1);
+ return (1);
lip += 3;
lm += 3;
ld += 3;
@@ -2276,7 +2276,7 @@ ipf_check_ipf(fr_info_t *fin, frentry_t *fr, int portcmp)
}
i ^= (fr->fr_flags & FR_NOTDSTIP) >> 7;
if (i != 0)
- return(1);
+ return (1);
/*
* IP addresses matched. The next 32bits contains:
* mast of old IP header security & authentication bits.
@@ -2319,7 +2319,7 @@ ipf_check_ipf(fr_info_t *fin, frentry_t *fr, int portcmp)
}
}
}
- return(i);
+ return (i);
}
@@ -2351,7 +2351,7 @@ ipf_scanlist(fr_info_t *fin, u_32_t pass)
* Do not allow nesting deeper than 16 levels.
*/
if (fin->fin_depth >= 16)
- return(pass);
+ return (pass);
fr = fin->fin_fr;
@@ -2359,7 +2359,7 @@ ipf_scanlist(fr_info_t *fin, u_32_t pass)
* If there are no rules in this list, return now.
*/
if (fr == NULL)
- return(pass);
+ return (pass);
skip = 0;
portcmp = 0;
@@ -2576,7 +2576,7 @@ ipf_scanlist(fr_info_t *fin, u_32_t pass)
}
}
fin->fin_depth--;
- return(pass);
+ return (pass);
}
@@ -2616,7 +2616,7 @@ ipf_acctpkt(fr_info_t *fin, u_32_t *passp)
bcopy(group, fin->fin_group, FR_GROUPLEN);
fin->fin_rule = rulen;
}
- return(NULL);
+ return (NULL);
}
@@ -2719,7 +2719,7 @@ ipf_firewall(fr_info_t *fin, u_32_t *passp)
fr = fin->fin_fr;
*passp = pass;
- return(fr);
+ return (fr);
}
@@ -2786,14 +2786,14 @@ ipf_check(void *ctx, ip_t *ip, int hlen, struct ifnet *ifp, int out
# ifdef __sparc
if ((u_int)ip & 0x3)
- return(2);
+ return (2);
# endif
# else
SPL_INT(s);
# endif
if (softc->ipf_running <= 0) {
- return(0);
+ return (0);
}
bzero((char *)fin, sizeof(*fin));
@@ -3182,9 +3182,9 @@ finished:
#ifdef _KERNEL
if (FR_ISPASS(pass))
- return(0);
+ return (0);
LBUMP(ipf_stats[out].fr_blocked[fin->fin_reason]);
- return(fin->fin_error);
+ return (fin->fin_error);
#else /* _KERNEL */
if (*mp != NULL)
(*mp)->mb_ifp = fin->fin_ifp;
@@ -3192,33 +3192,33 @@ finished:
FR_VERBOSE(("fin_flx %#x pass %#x ", fin->fin_flx, pass));
/*if ((pass & FR_CMDMASK) == (softc->ipf_pass & FR_CMDMASK))*/
if ((pass & FR_NOMATCH) != 0)
- return(1);
+ return (1);
if ((pass & FR_RETMASK) != 0)
switch (pass & FR_RETMASK)
{
case FR_RETRST :
- return(3);
+ return (3);
case FR_RETICMP :
- return(4);
+ return (4);
case FR_FAKEICMP :
- return(5);
+ return (5);
}
switch (pass & FR_CMDMASK)
{
case FR_PASS :
- return(0);
+ return (0);
case FR_BLOCK :
- return(-1);
+ return (-1);
case FR_AUTH :
- return(-2);
+ return (-2);
case FR_ACCOUNT :
- return(-3);
+ return (-3);
case FR_PREAUTH :
- return(-4);
+ return (-4);
}
- return(2);
+ return (2);
#endif /* _KERNEL */
}
@@ -3277,7 +3277,7 @@ logit:
*passp = pass;
}
- return(fin->fin_fr);
+ return (fin->fin_fr);
}
#endif /* IPFILTER_LOG */
@@ -3309,7 +3309,7 @@ ipf_cksum(u_short *addr, int len)
*/
sum = (sum >> 16) + (sum & 0xffff); /* add hi 16 to low 16 */
sum += (sum >> 16); /* add carry */
- return(u_short)(~sum);
+ return (u_short)(~sum);
}
@@ -3372,9 +3372,9 @@ fr_cksum(fr_info_t *fin, ip_t *ip, int l4proto, void *l4hdr)
ip6 = (ip6_t *)ip;
off = ((caddr_t)ip6 - m->m_data) + sizeof(struct ip6_hdr);
int len = ntohs(ip6->ip6_plen) - (off - sizeof(*ip6));
- return(ipf_pcksum6(m, ip6, off, len));
+ return (ipf_pcksum6(m, ip6, off, len));
} else {
- return(0xffff);
+ return (0xffff);
}
#endif
@@ -3408,7 +3408,7 @@ fr_cksum(fr_info_t *fin, ip_t *ip, int l4proto, void *l4hdr)
sum2 = ipf_pcksum(fin, off, sum);
if (csump != NULL)
*csump = sumsave;
- return(sum2);
+ return (sum2);
}
@@ -3445,7 +3445,7 @@ ipf_findgroup(ipf_main_softc_t *softc, char *group, minor_t unit, int set,
}
if (fgpp != NULL)
*fgpp = fgp;
- return(fg);
+ return (fg);
}
@@ -3472,10 +3472,10 @@ ipf_group_add(ipf_main_softc_t *softc, char *group, void *head, u_32_t flags,
u_32_t gflags;
if (group == NULL)
- return(NULL);
+ return (NULL);
if (unit == IPL_LOGIPF && *group == '\0')
- return(NULL);
+ return (NULL);
fgp = NULL;
gflags = flags & FR_INOUT;
@@ -3487,9 +3487,9 @@ ipf_group_add(ipf_main_softc_t *softc, char *group, void *head, u_32_t flags,
if (fg->fg_flags == 0)
fg->fg_flags = gflags;
else if (gflags != fg->fg_flags)
- return(NULL);
+ return (NULL);
fg->fg_ref++;
- return(fg);
+ return (fg);
}
KMALLOC(fg, frgroup_t *);
@@ -3503,7 +3503,7 @@ ipf_group_add(ipf_main_softc_t *softc, char *group, void *head, u_32_t flags,
fg->fg_set = &softc->ipf_groups[unit][set];
*fgp = fg;
}
- return(fg);
+ return (fg);
}
@@ -3570,7 +3570,7 @@ ipf_group_flush(ipf_main_softc_t *softc, frgroup_t *group)
(void) ipf_flushlist(softc, &gone, &group->fg_start);
- return(gone);
+ return (gone);
}
@@ -3594,12 +3594,12 @@ ipf_getrulen(ipf_main_softc_t *softc, int unit, char *group, u_32_t n)
fg = ipf_findgroup(softc, group, unit, softc->ipf_active, NULL);
if (fg == NULL)
- return(NULL);
+ return (NULL);
for (fr = fg->fg_start; fr && n; fr = fr->fr_next, n--)
;
if (n != 0)
- return(NULL);
- return(fr);
+ return (NULL);
+ return (fr);
}
@@ -3657,7 +3657,7 @@ ipf_flushlist(ipf_main_softc_t *softc, int *nfreedp, frentry_t **listp)
freed++;
}
*nfreedp += freed;
- return(freed);
+ return (freed);
}
@@ -3703,7 +3703,7 @@ ipf_flush(ipf_main_softc_t *softc, minor_t unit, int flags)
if (tmp >= 0)
flushed += tmp;
}
- return(flushed);
+ return (flushed);
}
@@ -3753,7 +3753,7 @@ ipf_flush_groups(ipf_main_softc_t *softc, frgroup_t **grhead, int flags)
if (removed == 0)
fgp = &fg->fg_next;
}
- return(flushed);
+ return (flushed);
}
@@ -3781,7 +3781,7 @@ memstr(const char *src, char *dst, size_t slen, size_t dlen)
dst++;
dlen--;
}
- return(s);
+ return (s);
}
/* ------------------------------------------------------------------------ */
/* Function: ipf_fixskip */
@@ -3844,8 +3844,8 @@ count4bits(u_32_t ip)
ipn++;
}
if (ipn == ip)
- return(cnt);
- return(-1);
+ return (cnt);
+ return (-1);
}
@@ -3872,7 +3872,7 @@ count6bits(u_32_t *msk)
if (j & 0x80000000)
i++;
}
- return(i);
+ return (i);
}
# endif
#endif /* _KERNEL */
@@ -3981,7 +3981,7 @@ ipf_synclist(ipf_main_softc_t *softc, frentry_t *fr, void *ifp)
&fr->fr_dstfunc);
}
}
- return(0);
+ return (0);
unwind:
for (frt = start; frt != fr; fr = fr->fr_next) {
@@ -3994,7 +3994,7 @@ unwind:
ipf_lookup_deref(softc, frt->fr_dsttype,
frt->fr_dstptr);
}
- return(error);
+ return (error);
}
@@ -4035,7 +4035,7 @@ ipf_sync(ipf_main_softc_t *softc, void *ifp)
}
RWLOCK_EXIT(&softc->ipf_mutex);
- return(0);
+ return (0);
}
@@ -4064,7 +4064,7 @@ copyinptr(ipf_main_softc_t *softc, void *src, void *dst, size_t size)
#if SOLARIS
error = COPYIN(src, &ca, sizeof(ca));
if (error != 0)
- return(error);
+ return (error);
#else
bcopy(src, (caddr_t)&ca, sizeof(ca));
#endif
@@ -4073,7 +4073,7 @@ copyinptr(ipf_main_softc_t *softc, void *src, void *dst, size_t size)
IPFERROR(3);
error = EFAULT;
}
- return(error);
+ return (error);
}
@@ -4100,7 +4100,7 @@ copyoutptr(ipf_main_softc_t *softc, void *src, void *dst, size_t size)
IPFERROR(4);
error = EFAULT;
}
- return(error);
+ return (error);
}
@@ -4120,12 +4120,12 @@ ipf_lock(caddr_t data, int *lockp)
err = BCOPYIN(data, &arg, sizeof(arg));
if (err != 0)
- return(EFAULT);
+ return (EFAULT);
err = BCOPYOUT(lockp, data, sizeof(*lockp));
if (err != 0)
- return(EFAULT);
+ return (EFAULT);
*lockp = arg;
- return(0);
+ return (0);
}
@@ -4275,9 +4275,9 @@ ipf_matchicmpqueryreply(int v, icmpinfo_t *ic, icmphdr_t *icmp, int rev)
if ((!rev && (icmp->icmp_type == ictype)) ||
(rev && (icmpreplytype4[ictype] == icmp->icmp_type))) {
if (icmp->icmp_type != ICMP_ECHOREPLY)
- return(1);
+ return (1);
if (icmp->icmp_id == ic->ici_id)
- return(1);
+ return (1);
}
}
#ifdef USE_INET6
@@ -4285,13 +4285,13 @@ ipf_matchicmpqueryreply(int v, icmpinfo_t *ic, icmphdr_t *icmp, int rev)
if ((!rev && (icmp->icmp_type == ictype)) ||
(rev && (icmpreplytype6[ictype] == icmp->icmp_type))) {
if (icmp->icmp_type != ICMP6_ECHO_REPLY)
- return(1);
+ return (1);
if (icmp->icmp_id == ic->ici_id)
- return(1);
+ return (1);
}
}
#endif
- return(0);
+ return (0);
}
@@ -4332,14 +4332,14 @@ ipf_rule_compare(frentry_t *fr1, frentry_t *fr2)
int i;
if (fr1->fr_cksum != fr2->fr_cksum)
- return(1);
+ return (1);
if (fr1->fr_size != fr2->fr_size)
- return(2);
+ return (2);
if (fr1->fr_dsize != fr2->fr_dsize)
- return(3);
+ return (3);
if (bcmp((char *)&fr1->fr_func, (char *)&fr2->fr_func, FR_CMPSIZ)
!= 0)
- return(4);
+ return (4);
/*
* XXX: There is still a bug here as different rules with the
* the same interfaces but in a different order will compare
@@ -4357,22 +4357,22 @@ ipf_rule_compare(frentry_t *fr1, frentry_t *fr2)
* referenced by the same rule doesn't work anyway.
*/
if (IPF_IFNAMES_DIFFERENT(fr_ifnames[i]))
- return(5);
+ return (5);
}
if (IPF_FRDEST_DIFFERENT(fr_tif))
- return(6);
+ return (6);
if (IPF_FRDEST_DIFFERENT(fr_rif))
- return(7);
+ return (7);
if (IPF_FRDEST_DIFFERENT(fr_dif))
- return(8);
+ return (8);
if (!fr1->fr_data && !fr2->fr_data)
- return(0); /* move along, nothing to see here */
+ return (0); /* move along, nothing to see here */
if (fr1->fr_data && fr2->fr_data) {
if (bcmp(fr1->fr_caddr, fr2->fr_caddr, fr1->fr_dsize) == 0)
- return(0); /* same */
+ return (0); /* same */
}
- return(9);
+ return (9);
}
@@ -4416,23 +4416,23 @@ frrequest(ipf_main_softc_t *softc, int unit, ioctlcmd_t req, caddr_t data,
bzero(fp, sizeof(frd));
error = ipf_inobj(softc, data, NULL, fp, IPFOBJ_FRENTRY);
if (error) {
- return(error);
+ return (error);
}
if ((fp->fr_type & FR_T_BUILTIN) != 0) {
IPFERROR(6);
- return(EINVAL);
+ return (EINVAL);
}
KMALLOCS(f, frentry_t *, fp->fr_size);
if (f == NULL) {
IPFERROR(131);
- return(ENOMEM);
+ return (ENOMEM);
}
bzero(f, fp->fr_size);
error = ipf_inobjsz(softc, data, f, IPFOBJ_FRENTRY,
fp->fr_size);
if (error) {
KFREES(f, fp->fr_size);
- return(error);
+ return (error);
}
fp = f;
@@ -4452,7 +4452,7 @@ frrequest(ipf_main_softc_t *softc, int unit, ioctlcmd_t req, caddr_t data,
fp = (frentry_t *)data;
if ((fp->fr_type & FR_T_BUILTIN) == 0) {
IPFERROR(7);
- return(EINVAL);
+ return (EINVAL);
}
fp->fr_flags &= ~FR_COPIED;
}
@@ -4918,7 +4918,7 @@ frrequest(ipf_main_softc_t *softc, int unit, ioctlcmd_t req, caddr_t data,
KFREES(fp, fp->fr_size);
}
RWLOCK_EXIT(&softc->ipf_mutex);
- return(error);
+ return (error);
}
if (f == NULL) {
@@ -5073,7 +5073,7 @@ donenolock:
}
KFREES(fp, fp->fr_size);
}
- return(error);
+ return (error);
}
@@ -5208,7 +5208,7 @@ ipf_findlookup(ipf_main_softc_t *softc, int unit, frentry_t *fr,
break;
}
- return(ptr);
+ return (ptr);
}
@@ -5237,7 +5237,7 @@ ipf_funcinit(ipf_main_softc_t *softc, frentry_t *fr)
err = (*ft->ipfu_init)(softc, fr);
break;
}
- return(err);
+ return (err);
}
@@ -5279,8 +5279,8 @@ ipf_findfunc(ipfunc_t funcptr)
for (ft = ipf_availfuncs; ft->ipfu_addr != NULL; ft++)
if (ft->ipfu_addr == funcptr)
- return(funcptr);
- return(NULL);
+ return (funcptr);
+ return (NULL);
}
@@ -5303,7 +5303,7 @@ ipf_resolvefunc(ipf_main_softc_t *softc, void *data)
error = BCOPYIN(data, &res, sizeof(res));
if (error != 0) {
IPFERROR(123);
*** 11711 LINES SKIPPED ***