git: e7611b7531c4 - main - ctld: Fix sign bug validating kernel ports when reloading config

From: John Baldwin <jhb_at_FreeBSD.org>
Date: Fri, 11 Apr 2025 14:03:54 UTC
The branch main has been updated by jhb:

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

commit e7611b7531c4fc6d26d36faad79088f277af78f0
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2025-04-11 13:59:24 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2025-04-11 13:59:24 +0000

    ctld: Fix sign bug validating kernel ports when reloading config
    
    When I switched new_pports_from_conf to return bool instead of an
    inverted int, I missed updating the call after reloading the config
    file due to SIGHUP.
    
    Fixes:          450a84c292ae ("ctld: Some bool-related cleanups")
    Sponsored by:   Chelsio Communications
    Differential Revision:  https://reviews.freebsd.org/D49641
---
 usr.sbin/ctld/ctld.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/usr.sbin/ctld/ctld.cc b/usr.sbin/ctld/ctld.cc
index f07ee7b34cd7..03a298146f4a 100644
--- a/usr.sbin/ctld/ctld.cc
+++ b/usr.sbin/ctld/ctld.cc
@@ -2601,7 +2601,7 @@ main(int argc, char **argv)
 			if (tmpconf == NULL) {
 				log_warnx("configuration error, "
 				    "continuing with old configuration");
-			} else if (new_pports_from_conf(tmpconf, &kports)) {
+			} else if (!new_pports_from_conf(tmpconf, &kports)) {
 				log_warnx("Error associating physical ports, "
 				    "continuing with old configuration");
 				conf_delete(tmpconf);