lib/, po/, src/: get_uid(): Move function to "atoi/getnum.h"

Implement it as an inline function, and add restrict and ATTR_STRING() and ATTR_ACCESS() as appropriate. Signed-off-by: Alejandro Colomar <[email protected]>
shadow-maint · Mar 15, 2024 · 2769280 · 2769280
1 parent ae87a7e
commit 2769280
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 23 deletions.
diff --git a/lib/Makefile.am b/lib/Makefile.am
@@ -74,7 +74,6 @@ libshadow_la_SOURCES = \
 	find_new_sub_uids.c \
 	fputsx.c \
 	get_pid.c \
-	get_uid.c \
 	getdate.h \
 	getdate.y \
 	getdef.c \

diff --git a/lib/atoi/getnum.c b/lib/atoi/getnum.c
@@ -13,3 +13,4 @@
 extern inline int get_fd(const char *restrict fdstr, int *restrict fd);
 extern inline int get_gid(const char *restrict gidstr, gid_t *restrict gid);
 extern inline int get_pid(const char *restrict pidstr, pid_t *restrict pid);
+extern inline int get_uid(const char *restrict uidstr, uid_t *restrict uid);
diff --git a/lib/atoi/getnum.h b/lib/atoi/getnum.h
@@ -24,6 +24,8 @@ ATTR_STRING(1) ATTR_ACCESS(write_only, 2)
 inline int get_gid(const char *restrict gidstr, gid_t *restrict gid);
 ATTR_STRING(1) ATTR_ACCESS(write_only, 2)
 inline int get_pid(const char *restrict pidstr, pid_t *restrict pid);
+ATTR_STRING(1) ATTR_ACCESS(write_only, 2)
+inline int get_uid(const char *restrict uidstr, uid_t *restrict uid);
 
 
 inline int
@@ -47,4 +49,11 @@ get_pid(const char *restrict pidstr, pid_t *restrict pid)
 }
 
 
+inline int
+get_uid(const char *restrict uidstr, uid_t *restrict uid)
+{
+	return a2i(uid_t, uid, uidstr, NULL, 10, type_min(uid_t), type_max(uid_t));
+}
+
+
 #endif  // include guard
diff --git a/lib/get_uid.c b/lib/get_uid.c
diff --git a/lib/prototypes.h b/lib/prototypes.h
@@ -157,9 +157,6 @@ extern int getrange (const char *range,
 /* gettime.c */
 extern time_t gettime (void);
 
-/* get_uid.c */
-extern int get_uid (const char *uidstr, uid_t *uid);
-
 /* fputsx.c */
 ATTR_ACCESS(write_only, 1, 2)
 extern /*@null@*/char *fgetsx(/*@returned@*/char *restrict, int, FILE *restrict);

diff --git a/po/POTFILES.in b/po/POTFILES.in
@@ -22,7 +22,6 @@ lib/find_new_sub_gids.c
 lib/find_new_sub_uids.c
 lib/find_new_uid.c
 lib/fputsx.c
-lib/get_uid.c
 lib/getdef.c
 lib/getgr_nam_gid.c
 lib/getrange.c

diff --git a/src/usermod.c b/src/usermod.c
@@ -34,6 +34,7 @@
 
 #include "alloc.h"
 #include "atoi/a2i.h"
+#include "atoi/getnum.h"
 #include "atoi/str2i.h"
 #include "chkname.h"
 #include "defines.h"