From: Alan T. DeKok <aland@freeradius.org>
Date: Fri, 6 Mar 2026 01:25:01 +0000 (-0500)
Subject: check for end pointer, too
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=ef323d039971ea7351efaa58b199ef990486eaa5;p=thirdparty%2Ffreeradius-server.git

check for end pointer, too
---

diff --git a/src/bin/radlock.c b/src/bin/radlock.c
index 3f994ba7751..93dc05260b7 100644
--- a/src/bin/radlock.c
+++ b/src/bin/radlock.c
@@ -90,6 +90,7 @@ int main(int argc, char *argv[])
 	int			c;
 	fr_radlock_action_t	action;
 	char const		*file;
+	char			*end;
 	uid_t			uid = geteuid();
 	bool			uid_set = false;
 	gid_t			gid = getegid();
@@ -129,8 +130,8 @@ int main(int argc, char *argv[])
 			break;
 
 		case 'm':
-			mode = strtol(optarg, NULL, 0);	/* 0 base plus 0 prefix = octal */
-			if (errno == EINVAL) {
+			mode = strtol(optarg, &end, 0);	/* 0 base plus 0 prefix = octal */
+			if (*end || (end == optarg)) {
 				fr_perror("radlock - Bad mode value");
 				EXIT_WITH_FAILURE;
 			}