From 1dd1e41fc2503fc9e77df4a91ec4e1dda5bea0be Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ermal=20Lu=E7i?= <eri@pfsense.org>
Date: Mon, 1 Feb 2010 10:45:02 +0000
Subject: [PATCH] Ticket #331. For now skip virtual interfaces on loginterface
 directive. This is contrary to what OpenBSD supports and is being discussed
 on how to proceed.

---
 etc/inc/filter.inc | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/etc/inc/filter.inc b/etc/inc/filter.inc
index 152cbb3df6..f1b8f0be06 100644
--- a/etc/inc/filter.inc
+++ b/etc/inc/filter.inc
@@ -2252,9 +2252,15 @@ function filter_setup_logging_interfaces() {
 		echo "filter_setup_logging_interfaces() being called $mt\n";
 	}
 	$rules = "";
-	foreach ($FilterIflist as $ifdescr => $ifname) {
-		/* do not work with tun interfaces */
-		$rules .= "set loginterface {$ifname['if']}\n";
+	foreach ($FilterIflist as $ifdescr => $ifcfg) {
+		/* 
+		 * XXX: This should be cleared out after a discussion
+		 * between pf(4) devs is cleared out. This breaks 
+		 * compatibility with OpenBSD.
+		 */ 
+		if(isset($ifcfg['virtual']))
+                        continue;
+		$rules .= "set loginterface {$ifcfg['if']}\n";
 	}
 	return $rules;
 }