diff options
| author | Kristof Provost <kp@FreeBSD.org> | 2026-07-21 13:10:16 +0000 |
|---|---|---|
| committer | Kristof Provost <kp@FreeBSD.org> | 2026-07-29 07:21:15 +0000 |
| commit | d68de8c2cd953a8df9db63ae50a9eb3e7f6d96dc (patch) | |
| tree | 2cee7dbe61ff066c7924b86c6fe461ec9c4a61e7 | |
| parent | e7381649048feb2e30f2aef2dac0e4924e7d815a (diff) | |
authpf(8) read_config() should chop off trailing white space
if administrator mistakenly types into configuration file
anchor=authpf_test
where 'authpf_test' is followed by white space, the authpf(8)
is going to use anchor 'authpf_test ' instead of the 'authpf_test'
which is defined in pf.conf(5) as 'anchor authpf_test/*'
issue kindly reported and patch submitted by
Avinash Duduskar <avinash.duduskar (_at_) gmail (_dot_) com>
OK sashan@
PR: 296958
MFC after: 1 week
Obtained from: OpenBSD, sashan <sashan@openbsd.org>, 2d12a8e44d
Sponsored by: Rubicon Communications, LLC ("Netgate")
(cherry picked from commit 04f25ef716f74d6bb7941750091c6cb4b51d0b4d)
| -rw-r--r-- | contrib/pf/authpf/authpf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/contrib/pf/authpf/authpf.c b/contrib/pf/authpf/authpf.c index 14d3d41b6e21..b835737c5ee2 100644 --- a/contrib/pf/authpf/authpf.c +++ b/contrib/pf/authpf/authpf.c @@ -404,8 +404,8 @@ read_config(FILE *f) if (ap != &pair[2]) goto parse_error; - tp = pair[1] + strlen(pair[1]); - while ((*tp == ' ' || *tp == '\t') && tp >= pair[1]) + tp = pair[1] + strlen(pair[1]) - 1; + while (tp >= pair[1] && (*tp == ' ' || *tp == '\t')) *tp-- = '\0'; if (strcasecmp(pair[0], "anchor") == 0) { |