Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 16 Feb 2015 23:50:54 +0000 (UTC)
From:      Gleb Smirnoff <glebius@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r278874 - in head/sys: net netpfil/pf
Message-ID:  <201502162350.t1GNosXm026325@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: glebius
Date: Mon Feb 16 23:50:53 2015
New Revision: 278874
URL: https://svnweb.freebsd.org/changeset/base/278874

Log:
  - Improve INET/INET6 scope.
  - style(9) declarations.
  - Make couple of local functions static.

Modified:
  head/sys/net/pfvar.h
  head/sys/netpfil/pf/pf_norm.c

Modified: head/sys/net/pfvar.h
==============================================================================
--- head/sys/net/pfvar.h	Mon Feb 16 23:29:56 2015	(r278873)
+++ head/sys/net/pfvar.h	Mon Feb 16 23:50:53 2015	(r278874)
@@ -1567,10 +1567,14 @@ void				pf_free_rule(struct pf_rule *);
 
 #ifdef INET
 int	pf_test(int, struct ifnet *, struct mbuf **, struct inpcb *);
+int	pf_normalize_ip(struct mbuf **, int, struct pfi_kif *, u_short *,
+	    struct pf_pdesc *);
 #endif /* INET */
 
 #ifdef INET6
 int	pf_test6(int, struct ifnet *, struct mbuf **, struct inpcb *);
+int	pf_normalize_ip6(struct mbuf **, int, struct pfi_kif *, u_short *,
+	    struct pf_pdesc *);
 void	pf_poolmask(struct pf_addr *, struct pf_addr*,
 	    struct pf_addr *, struct pf_addr *, u_int8_t);
 void	pf_addr_inc(struct pf_addr *, sa_family_t);
@@ -1590,10 +1594,6 @@ int	pf_match_port(u_int8_t, u_int16_t, u
 
 void	pf_normalize_init(void);
 void	pf_normalize_cleanup(void);
-int	pf_normalize_ip(struct mbuf **, int, struct pfi_kif *, u_short *,
-	    struct pf_pdesc *);
-int	pf_normalize_ip6(struct mbuf **, int, struct pfi_kif *, u_short *,
-	    struct pf_pdesc *);
 int	pf_normalize_tcp(int, struct pfi_kif *, struct mbuf *, int, int, void *,
 	    struct pf_pdesc *);
 void	pf_normalize_tcp_cleanup(struct pf_state *);

Modified: head/sys/netpfil/pf/pf_norm.c
==============================================================================
--- head/sys/netpfil/pf/pf_norm.c	Mon Feb 16 23:29:56 2015	(r278873)
+++ head/sys/netpfil/pf/pf_norm.c	Mon Feb 16 23:50:53 2015	(r278874)
@@ -134,35 +134,32 @@ static int		 pf_frag_compare(struct pf_f
 static RB_PROTOTYPE(pf_frag_tree, pf_fragment, fr_entry, pf_frag_compare);
 static RB_GENERATE(pf_frag_tree, pf_fragment, fr_entry, pf_frag_compare);
 
-/* Private prototypes */
-static void		 pf_free_fragment(struct pf_fragment *);
-static void		 pf_remove_fragment(struct pf_fragment *);
-static int		 pf_normalize_tcpopt(struct pf_rule *, struct mbuf *,
-			    struct tcphdr *, int, sa_family_t);
+static void	pf_free_fragment(struct pf_fragment *);
+static void	pf_remove_fragment(struct pf_fragment *);
+static int	pf_normalize_tcpopt(struct pf_rule *, struct mbuf *,
+		    struct tcphdr *, int, sa_family_t);
+struct pf_frent	*pf_create_fragment(u_short *);
+static struct pf_fragment *
+		pf_find_fragment(struct pf_fragment_cmp *key,
+		    struct pf_frag_tree *tree);
+static struct pf_fragment *
+		pf_fillup_fragment(struct pf_fragment_cmp *, struct pf_frent *,			    u_short *);
+static int	pf_isfull_fragment(struct pf_fragment *);
+struct mbuf	*pf_join_fragment(struct pf_fragment *);
 #ifdef INET
-static void		 pf_scrub_ip(struct mbuf **, u_int32_t, u_int8_t,
-			    u_int8_t);
-static void		 pf_flush_fragments(void);
-static struct pf_fragment *pf_find_fragment(struct pf_fragment_cmp *key,
-			struct pf_frag_tree *tree);
-struct pf_frent		*pf_create_fragment(u_short *);
-static int		pf_reassemble(struct mbuf **, struct ip *, int,
-			    u_short *);
-static struct mbuf	*pf_fragcache(struct mbuf **, struct ip*,
-			    struct pf_fragment **, int, int, int *);
-static struct pf_fragment *pf_fillup_fragment(struct pf_fragment_cmp *,
-			    struct pf_frent *, u_short *);
-int			pf_isfull_fragment(struct pf_fragment *);
-struct mbuf		*pf_join_fragment(struct pf_fragment *);
-
-
-#endif /* INET */
+static void	pf_scrub_ip(struct mbuf **, uint32_t, uint8_t, uint8_t);
+static void	pf_flush_fragments(void);
+static int	pf_reassemble(struct mbuf **, struct ip *, int, u_short *);
+static struct mbuf *
+		pf_fragcache(struct mbuf **, struct ip*, struct pf_fragment **,
+		    int, int, int *);
+#endif	/* INET */
 #ifdef INET6
-int			pf_reassemble6(struct mbuf **, struct ip6_hdr *,
-			    struct ip6_frag *, uint16_t, uint16_t, int,
-			    u_short *);
-static void		 pf_scrub_ip6(struct mbuf **, u_int8_t);
-#endif
+static int	pf_reassemble6(struct mbuf **, struct ip6_hdr *,
+		    struct ip6_frag *, uint16_t, uint16_t, int, u_short *);
+static void	pf_scrub_ip6(struct mbuf **, uint8_t);
+#endif	/* INET6 */
+
 #define	DPFPRINTF(x) do {				\
 	if (V_pf_status.debug >= PF_DEBUG_MISC) {	\
 		printf("%s: ", __func__);		\
@@ -170,6 +167,7 @@ static void		 pf_scrub_ip6(struct mbuf *
 	}						\
 } while(0)
 
+#ifdef INET
 static void
 pf_ip2key(struct ip *ip, int dir, struct pf_fragment_cmp *key)
 {
@@ -181,6 +179,7 @@ pf_ip2key(struct ip *ip, int dir, struct
 	key->frc_id = ip->ip_id;
 	key->frc_direction = dir;
 }
+#endif	/* INET */
 
 void
 pf_normalize_init(void)
@@ -374,7 +373,6 @@ pf_remove_fragment(struct pf_fragment *f
 	}
 }
 
-#ifdef INET
 struct pf_frent *
 pf_create_fragment(u_short *reason)
 {
@@ -538,7 +536,7 @@ drop_fragment:
 	return (NULL);
 }
 
-int
+static int
 pf_isfull_fragment(struct pf_fragment *frag)
 {
 	struct pf_frent	*frent, *next;
@@ -603,7 +601,7 @@ pf_join_fragment(struct pf_fragment *fra
 	return (m);
 }
 
-#define FR_IP_OFF(fr)	((ntohs((fr)->fr_ip->ip_off) & IP_OFFMASK) << 3)
+#ifdef INET
 static int
 pf_reassemble(struct mbuf **m0, struct ip *ip, int dir, u_short *reason)
 {
@@ -670,9 +668,9 @@ pf_reassemble(struct mbuf **m0, struct i
 }
 
 #ifdef INET6
-int
+static int
 pf_reassemble6(struct mbuf **m0, struct ip6_hdr *ip6, struct ip6_frag *fraghdr,
-		uint16_t hdrlen, uint16_t extoff, int dir, u_short *reason)
+    uint16_t hdrlen, uint16_t extoff, int dir, u_short *reason)
 {
 	struct mbuf		*m = *m0;
 	struct pf_frent		*frent;
@@ -789,8 +787,7 @@ fail:
 	/* PF_DROP requires a valid mbuf *m0 in pf_test6(), will free later. */
 	return (PF_DROP);
 }
-
-#endif
+#endif	/* INET6 */
 
 static struct mbuf *
 pf_fragcache(struct mbuf **m0, struct ip *h, struct pf_fragment **frag, int mff,



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201502162350.t1GNosXm026325>