From: Catalina Pineros <catalina.pineros@inkbridge.io>
Date: Thu, 12 Mar 2026 17:42:48 +0000 (-0400)
Subject: fix whitespace stripping that never executes
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f27f6aa6767a39099639638700355002e4ed4fc7;p=thirdparty%2Ffreeradius-server.git

fix whitespace stripping that never executes

buff[2][slen] is set to '/0', which is not a whitespace character, so we need to check buff[2][slen - 1] to actually remove trailing whitespace
---

diff --git a/src/lib/server/cf_file.c b/src/lib/server/cf_file.c
index 98c54db982b..55f398a4c51 100644
--- a/src/lib/server/cf_file.c
+++ b/src/lib/server/cf_file.c
@@ -1607,9 +1607,9 @@ static CONF_ITEM *process_if(cf_stack_t *stack)
 	buff[2][slen] = '\0';
 
 	while (slen > 0) {
-		if (!isspace((uint8_t) buff[2][slen])) break;
+		if (!isspace((uint8_t) buff[2][slen - 1])) break;
 
-		buff[2][slen] = '\0';
+		buff[2][slen - 1] = '\0';
 		slen--;
 	}