src/: Use getl() instead of atoi(3)

atoi(3) easily triggers Undefined Behavior.  Replace it by getl(), which
is safe from that, and adds type safety too.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

src/free_subid_range.c

@@ -1,12 +1,16 @@
 /* SPDX-License-Identifier: BSD-3-Clause */
 
+
 #include <stdio.h>
 #include <unistd.h>
+
+#include "atoi/getlong.h"
 #include "subid.h"
 #include "stdlib.h"
 #include "prototypes.h"
 #include "shadowlog.h"
 
+
 /* Test program for the subid freeing routine */
 
 const char *Prog;
@@ -20,8 +24,9 @@ static void usage(void)
 
 int main(int argc, char *argv[])
 {
-	int c;
-	bool ok;
+	int   c;
+	bool  ok;
+	long  l;
 	struct subordinate_range range;
 	bool group = false;   // get subuids by default
 
@@ -39,8 +44,8 @@ int main(int argc, char *argv[])
 	if (argc < 3)
 		usage();
 	range.owner = argv[0];
-	range.start = atoi(argv[1]);
-	range.count = atoi(argv[2]);
+	getl(argv[1], &range.start);
+	getl(argv[2], &range.count);
 	if (group)
 		ok = subid_ungrant_gid_range(&range);
 	else

src/get_subid_owners.c

@@ -1,11 +1,15 @@
 /* SPDX-License-Identifier: BSD-3-Clause */
 
+
 #include <stdio.h>
+
+#include "atoi/getlong.h"
 #include "subid.h"
 #include "stdlib.h"
 #include "prototypes.h"
 #include "shadowlog.h"
 
+
 const char *Prog;
 
 static void usage(void)
@@ -18,21 +22,25 @@ static void usage(void)
 
 int main(int argc, char *argv[])
 {
-	int i, n;
-	uid_t *uids;
+	int    i, n;
+	long   l;
+	uid_t  *uids;
 
 	Prog = Basename (argv[0]);
 	log_set_progname(Prog);
 	log_set_logfd(stderr);
 	if (argc < 2) {
 		usage();
 	}
-	if (argc == 3 && strcmp(argv[1], "-g") == 0)
-		n = subid_get_gid_owners(atoi(argv[2]), &uids);
-	else if (argc == 2 && strcmp(argv[1], "-h") == 0)
+	if (argc == 3 && strcmp(argv[1], "-g") == 0) {
+		getl(argv[2], &l);
+		n = subid_get_gid_owners(l, &uids);
+	} else if (argc == 2 && strcmp(argv[1], "-h") == 0) {
 		usage();
-	else
-		n = subid_get_uid_owners(atoi(argv[1]), &uids);
+	} else {
+		getl(argv[1], &l);
+		n = subid_get_uid_owners(l, &uids);
+	}
 	if (n < 0) {
 		fprintf(stderr, "No owners found\n");
 		exit(1);

src/new_subid_range.c

@@ -2,11 +2,14 @@
 
 #include <stdio.h>
 #include <unistd.h>
+
+#include "atoi/getlong.h"
 #include "subid.h"
 #include "stdlib.h"
 #include "prototypes.h"
 #include "shadowlog.h"
 
+
 /* Test program for the subid creation routine */
 
 const char *Prog;
@@ -46,7 +49,7 @@ int main(int argc, char *argv[])
 	range.start = 0;
 	range.count = 65536;
 	if (argc > 1)
-		range.count = atoi(argv[1]);
+		getl(argv[1], &range.count);
 	if (group)
 		ok = subid_grant_gid_range(&range, !makenew);
 	else