carpdev : bad value?

Xin LI delphij at delphij.net
Wed Dec 30 17:05:26 UTC 2009


On 2009/12/30 08:06, kevin wrote:
>> I have committed a patch that makes pf w/DSR setup work a week ago but
>> have not yet MFC'ed it, the patch can be directly applied against
>> 8-STABLE, though.
>
> Would you be able to share the patch with me? I am on 7.2-RELEASE, however.

The patch should work cleanly on RELENG_7_2 (sys/net).

However, based on your usage I don't think this is appropriate, it's a 
workaround and I believe it would cause problem for your setup which 
rely on bridge...

Cheers,
-- 
Xin LI <delphij at delphij.net>	http://www.delphij.net/
FreeBSD - The Power to Serve!	       Live free or die
-------------- next part --------------
Index: if_ethersubr.c
===================================================================
--- if_ethersubr.c	(revision 201259)
+++ if_ethersubr.c	(working copy)
@@ -347,8 +347,18 @@
 	* Bridges require special output handling.
 	*/
 	if (ifp->if_bridge) {
+#if defined(INET) || defined(INET6)
+#ifdef DEV_CARP
+		if ((m->m_flags & ~(M_MCAST | M_BCAST)) == m->m_flags) {
+#endif
+#endif
 		BRIDGE_OUTPUT(ifp, m, error);
 		return (error);
+#if defined(INET) || defined(INET6)
+#ifdef DEV_CARP
+		}
+#endif
+#endif
 	}
 
 #ifdef DEV_CARP


More information about the freebsd-pf mailing list