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,
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to