Do proper error handling with strtonum

Checking for an error string is the proper step. Have an error string for consistency.
openssh · Jul 7, 2023 · 556f58e · 556f58e
1 parent 7e8800f
commit 556f58e
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/ssh-add.c b/ssh-add.c
@@ -788,6 +788,7 @@ main(int argc, char **argv)
 	extern char *optarg;
 	extern int optind;
 	int agent_fd;
+	const char *errstr = NULL;
 	char *pkcs11provider = NULL, *skprovider = NULL;
 	char **dest_constraint_strings = NULL, **hostkey_files = NULL;
 	int r, i, ch, deleting = 0, ret = 0, key_only = 0, do_download = 0;
@@ -863,16 +864,16 @@ main(int argc, char **argv)
 			confirm = 1;
 			break;
 		case 'm':
-			minleft = (u_int)strtonum(optarg, 1, UINT_MAX, NULL);
-			if (minleft == 0) {
+			minleft = (u_int)strtonum(optarg, 1, UINT_MAX, &errstr);
+			if (errstr) {
 				usage();
 				ret = 1;
 				goto done;
 			}
 			break;
 		case 'M':
-			maxsign = (u_int)strtonum(optarg, 1, UINT_MAX, NULL);
-			if (maxsign == 0) {
+			maxsign = (u_int)strtonum(optarg, 1, UINT_MAX, &errstr);
+			if (errstr) {
 				usage();
 				ret = 1;
 				goto done;