Expunge strtoll(3) and strtol(3) #896
Commits on May 27, 2024
-
lib/typetraits.h: Add macros that give information about a type
In the case of is_unsigned() and is_signed(), the natural thing would be to compare to 0: #define is_unsigned(x) (((typeof(x)) -1) > 0) #define is_signed(x) (((typeof(x)) -1) < 0) However, that would trigger -Wtype-limits, so we compare against 1, which silences that, and does the same job. Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
src/: Use str2[u]l() instead of atoi(3)
atoi(3) easily triggers Undefined Behavior. Replace it by str2[u]l(), which are safe from that, and add type safety too. Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
lib/get_gid.c: get_gid(): Reimplement in terms of a2i()
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
lib/, libsubid/, po/, src/: get_gid(): 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 <alx@kernel.org>
-
lib/: Don't open-code get_gid()
These functions were open-coding get_gid(). Use the actual function. Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
lib/get_pid.c: get_pid(): Reimplement in terms of a2i()
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
lib/: get_pid(): 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 <alx@kernel.org>
-
lib/atoi/getnum.[ch]: get_fd(): Add function for parsing a file descr…
…iptor from a string Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
lib/get_pid.c: get_pidfd_from_fd(): Don't open-code get_fd()
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
src/usermod.c: getulong_range(): Reimplement in terms of a2ul()
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
lib/get_uid.c: get_uid(): Reimplement in terms of a2i()
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
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 <alx@kernel.org>
-
lib/limits.c: setrlimit_value(): Reimplement in terms of a2i()
Signed-off-by: Alejandro Colomar <alx@kernel.org>
-
src/usermod.c: Fix const correctness
Now that we use liba2i's const-generic macros, we can (and must) use a 'const char **' endp where the input string is 'const char *'. Signed-off-by: Alejandro Colomar <alx@kernel.org>
