Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 24 Mar 2026 06:03:57 +0000
From:      Kristof Provost <kp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 66d66dd0f6f8 - main - pfctl: always warn if a duplicate rule was detected
Message-ID:  <69c2294d.3e058.4f1c6b50@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch main has been updated by kp:

URL: https://cgit.FreeBSD.org/src/commit/?id=66d66dd0f6f83926980fc1d68dd366c0057350c5

commit 66d66dd0f6f83926980fc1d68dd366c0057350c5
Author:     Kristof Provost <kp@FreeBSD.org>
AuthorDate: 2026-03-12 14:24:42 +0000
Commit:     Kristof Provost <kp@FreeBSD.org>
CommitDate: 2026-03-24 04:27:20 +0000

    pfctl: always warn if a duplicate rule was detected
    
    MFC after:      1 week
    Sponsored by:   Rubicon Communications, LLC ("Netgate")
---
 sbin/pfctl/pfctl.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sbin/pfctl/pfctl.c b/sbin/pfctl/pfctl.c
index 5a4668416b5b..a7bba4055b06 100644
--- a/sbin/pfctl/pfctl.c
+++ b/sbin/pfctl/pfctl.c
@@ -2615,13 +2615,13 @@ pfctl_load_rule(struct pfctl *pf, char *path, struct pfctl_rule *r, int depth)
 		}
 	}
 
-	if (pf->opts & PF_OPT_VERBOSE) {
+	if (pf->opts & PF_OPT_VERBOSE || was_present) {
 		INDENT(depth, !(pf->opts & PF_OPT_VERBOSE2));
 		print_rule(r, name,
 		    pf->opts & PF_OPT_VERBOSE2,
 		    pf->opts & PF_OPT_NUMERIC);
 		if (was_present)
-			printf(" -- rule was already present");
+			printf(" -- rule was already present\n");
 	}
 	path[len] = '\0';
 	pfctl_clear_pool(&r->rdr);


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?69c2294d.3e058.4f1c6b50>