Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 19 Nov 2024 00:26:47 GMT
From:      Cy Schubert <cy@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: c6385ba2c9ff - stable/14 - ipfilter: Print the pool name when adding a poolnode fails
Message-ID:  <202411190026.4AJ0Ql4C063691@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/14 has been updated by cy:

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

commit c6385ba2c9ff20d6cdc4ff673ff993a1adbd49a2
Author:     Cy Schubert <cy@FreeBSD.org>
AuthorDate: 2024-11-07 01:26:37 +0000
Commit:     Cy Schubert <cy@FreeBSD.org>
CommitDate: 2024-11-19 00:26:04 +0000

    ipfilter: Print the pool name when adding a poolnode fails
    
    Printing the pool node (IP address) only leads to speculation which
    pool the failed add occurred. This is especially useful when the same
    IP address is listed in multiple pools.
    
    (cherry picked from commit 470c680cfbf5f6cd9ac392b210b97b41d02cd969)
---
 sbin/ipf/libipf/load_poolnode.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sbin/ipf/libipf/load_poolnode.c b/sbin/ipf/libipf/load_poolnode.c
index 8d65f019da43..7a01da9df94b 100644
--- a/sbin/ipf/libipf/load_poolnode.c
+++ b/sbin/ipf/libipf/load_poolnode.c
@@ -54,8 +54,8 @@ load_poolnode(int role, char *name, ip_pool_node_t *node, int ttl,
 		if ((opts & OPT_DONOTHING) == 0) {
 			char msg[255];
 
-			snprintf(msg, sizeof(msg), "%s pool node(%s/", what,
-				inet_ntoa(pn.ipn_addr.adf_addr.in4));
+			snprintf(msg, sizeof(msg), "%s pool(%s) node(%s/", what,
+				name, inet_ntoa(pn.ipn_addr.adf_addr.in4));
 			strlcat(msg, inet_ntoa(pn.ipn_mask.adf_addr.in4), sizeof(msg));
 			return (ipf_perror_fd(pool_fd(), iocfunc, msg));
 		}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202411190026.4AJ0Ql4C063691>