Author: kp
Date: Tue Jan 15 08:59:51 2019
New Revision: 343041
URL: https://svnweb.freebsd.org/changeset/base/343041

Log:
  pf: silence a runtime warning
  
  Sometimes, for negated tables, pf can log 'pfr_update_stats: assertion 
failed'.
  This warning does not clarify anything for users, so silence it, just as
  OpenBSD has.
  
  PR:           234874
  MFC after:    1 week

Modified:
  head/sys/netpfil/pf/pf_table.c

Modified: head/sys/netpfil/pf/pf_table.c
==============================================================================
--- head/sys/netpfil/pf/pf_table.c      Tue Jan 15 06:52:34 2019        
(r343040)
+++ head/sys/netpfil/pf/pf_table.c      Tue Jan 15 08:59:51 2019        
(r343041)
@@ -51,6 +51,8 @@ __FBSDID("$FreeBSD$");
 #include <net/vnet.h>
 #include <net/pfvar.h>
 
+#define DPFPRINTF(n, x) if (V_pf_status.debug >= (n)) printf x
+
 #define        ACCEPT_FLAGS(flags, oklist)             \
        do {                                    \
                if ((flags & ~(oklist)) &       \
@@ -1992,7 +1994,8 @@ pfr_update_stats(struct pfr_ktable *kt, struct pf_addr
        }
        if ((ke == NULL || ke->pfrke_not) != notrule) {
                if (op_pass != PFR_OP_PASS)
-                       printf("pfr_update_stats: assertion failed.\n");
+                       DPFPRINTF(PF_DEBUG_URGENT,
+                           ("pfr_update_stats: assertion failed.\n"));
                op_pass = PFR_OP_XPASS;
        }
        kt->pfrkt_packets[dir_out][op_pass]++;
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to