Author: gnn
Date: Sat May  9 19:43:48 2015
New Revision: 282689
URL: https://svnweb.freebsd.org/changeset/base/282689

Log:
  MFC: 281558
  
  Minor change to the macros to make sure that if an AF is passed that is 
neither AF_INET6 nor AF_INET that we don't touch random bits of memory.

Modified:
  stable/10/sys/net/pfvar.h
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/net/pfvar.h
==============================================================================
--- stable/10/sys/net/pfvar.h   Sat May  9 19:36:30 2015        (r282688)
+++ stable/10/sys/net/pfvar.h   Sat May  9 19:43:48 2015        (r282689)
@@ -192,21 +192,20 @@ extern struct rwlock pf_rules_lock;
 
 #define PF_AEQ(a, b, c) \
        ((c == AF_INET && (a)->addr32[0] == (b)->addr32[0]) || \
-       ((a)->addr32[3] == (b)->addr32[3] && \
+       (c == AF_INET6 && (a)->addr32[3] == (b)->addr32[3] && \
        (a)->addr32[2] == (b)->addr32[2] && \
        (a)->addr32[1] == (b)->addr32[1] && \
        (a)->addr32[0] == (b)->addr32[0])) \
 
 #define PF_ANEQ(a, b, c) \
-       ((c == AF_INET && (a)->addr32[0] != (b)->addr32[0]) || \
-       ((a)->addr32[3] != (b)->addr32[3] || \
-       (a)->addr32[2] != (b)->addr32[2] || \
+       ((a)->addr32[0] != (b)->addr32[0] || \
        (a)->addr32[1] != (b)->addr32[1] || \
-       (a)->addr32[0] != (b)->addr32[0])) \
+       (a)->addr32[2] != (b)->addr32[2] || \
+       (a)->addr32[3] != (b)->addr32[3]) \
 
 #define PF_AZERO(a, c) \
        ((c == AF_INET && !(a)->addr32[0]) || \
-       (!(a)->addr32[0] && !(a)->addr32[1] && \
+       (c == AF_INET6 && !(a)->addr32[0] && !(a)->addr32[1] && \
        !(a)->addr32[2] && !(a)->addr32[3] )) \
 
 #define PF_MATCHA(n, a, m, b, f) \
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to