git: e4a2b19aa961 - main - pfctl: err(3) family functions should not be used with trailing \n

From: Kristof Provost <kp_at_FreeBSD.org>
Date: Mon, 02 Nov 2026 13:25:10 UTC
The branch main has been updated by kp:

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

commit e4a2b19aa961eca9775c5fde6512619d3141b1c6
Author:     Kristof Provost <kp@FreeBSD.org>
AuthorDate: 2026-11-02 10:32:14 +0000
Commit:     Kristof Provost <kp@FreeBSD.org>
CommitDate: 2026-11-02 12:40:02 +0000

    pfctl: err(3) family functions should not be used with trailing \n
    
    Obtained from:  OpenBSD, deraadt <deraadt@openbsd.org>, 3c04d8eee2
    Sponsored by:   Rubicon Communications, LLC ("Netgate")
---
 sbin/pfctl/parse.y | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sbin/pfctl/parse.y b/sbin/pfctl/parse.y
index 945955059d91..b9099fb603e8 100644
--- a/sbin/pfctl/parse.y
+++ b/sbin/pfctl/parse.y
@@ -7832,7 +7832,7 @@ mv_tables(struct pfctl *pf, struct pfr_ktablehead *ktables,
 				sz = snprintf(new_path, sizeof (new_path),
 				    "%s", a->path);
 			if (sz >= sizeof (new_path))
-				errx(1, "new path is too long for %s@%s\n",
+				errx(1, "new path is too long for %s@%s",
 				    kt->pfrkt_name, kt->pfrkt_anchor);
 
 			DBGPRINT("%s %s@%s -> %s@%s\n", __func__,
@@ -7850,7 +7850,7 @@ mv_tables(struct pfctl *pf, struct pfr_ktablehead *ktables,
 		SLIST_REMOVE_HEAD(&ukt_list, pfrukt_entry);
 		if (RB_INSERT(pfr_ktablehead, ktables,
 		    (struct pfr_ktable *)ukt) != NULL)
-			errx(1, "%s@%s exists already\n",
+			errx(1, "%s@%s exists already",
 			    ukt->pfrukt_name,
 			    ukt->pfrukt_anchor);
 	}