|
 |
7ca4a1 |
From ae0a9312c562985838fdd9845ef95fe61e8aa3de Mon Sep 17 00:00:00 2001
|
|
|
7ca4a1 |
From: Jakub Hrozek <jakub.hrozek@posteo.se>
|
|
|
7ca4a1 |
Date: Sun, 1 Apr 2018 10:57:22 +0200
|
|
|
7ca4a1 |
Subject: [PATCH 2/2] Watch for uint32_t overflows
|
|
|
7ca4a1 |
|
|
|
7ca4a1 |
Always use a function that we know will catch out-of-range values for UIDs and
|
|
|
7ca4a1 |
GIDs, which are currently unsigned 32-bit numbers everywhere, and which won't
|
|
|
7ca4a1 |
produce a result that'll silently be truncated if we store the result in a
|
|
|
7ca4a1 |
uid_t or gid_t.
|
|
|
7ca4a1 |
---
|
|
|
7ca4a1 |
nslcd/common.c | 28 ++++++++++++++++------------
|
|
|
7ca4a1 |
nslcd/common.h | 27 +++------------------------
|
|
|
7ca4a1 |
2 files changed, 19 insertions(+), 36 deletions(-)
|
|
|
7ca4a1 |
|
|
|
7ca4a1 |
diff --git a/nslcd/common.c b/nslcd/common.c
|
|
|
7ca4a1 |
index 60be7773d2c809f3177744ced0dd0ba90c86e820..de640b47806757e0bb2e704b3b79f1ecb18bbc45 100644
|
|
|
7ca4a1 |
--- a/nslcd/common.c
|
|
|
7ca4a1 |
+++ b/nslcd/common.c
|
|
|
7ca4a1 |
@@ -338,19 +338,23 @@ unsigned long int binsid2id(const char *binsid)
|
|
|
7ca4a1 |
((((unsigned long int)binsid[i + 3]) & 0xff) << 24);
|
|
|
7ca4a1 |
}
|
|
|
7ca4a1 |
|
|
|
7ca4a1 |
-#ifdef WANT_STRTOUI
|
|
|
7ca4a1 |
-/* provide a strtoui() implementation, similar to strtoul() but returning
|
|
|
7ca4a1 |
- an range-checked unsigned int instead */
|
|
|
7ca4a1 |
-unsigned int strtoui(const char *nptr, char **endptr, int base)
|
|
|
7ca4a1 |
+/* provide a strtoid() implementation, similar to strtoul() but returning
|
|
|
7ca4a1 |
+ an range-checked uint32_t instead */
|
|
|
7ca4a1 |
+unsigned int strtoid(const char *nptr,char **endptr,int base)
|
|
|
7ca4a1 |
{
|
|
|
7ca4a1 |
- unsigned long val;
|
|
|
7ca4a1 |
- val = strtoul(nptr, endptr, base);
|
|
|
7ca4a1 |
- if (val > UINT_MAX)
|
|
|
7ca4a1 |
+ long long val;
|
|
|
7ca4a1 |
+ /* use the fact that long long is 64-bit, even on 32-bit systems */
|
|
|
7ca4a1 |
+ val=strtoll(nptr,endptr,base);
|
|
|
7ca4a1 |
+ if (val>UINT32_MAX)
|
|
|
7ca4a1 |
{
|
|
|
7ca4a1 |
- errno = ERANGE;
|
|
|
7ca4a1 |
- return UINT_MAX;
|
|
|
7ca4a1 |
+ errno=ERANGE;
|
|
|
7ca4a1 |
+ return UINT32_MAX;
|
|
|
7ca4a1 |
}
|
|
|
7ca4a1 |
- /* If errno was set by strtoul, we'll pass it back as-is */
|
|
|
7ca4a1 |
- return (unsigned int)val;
|
|
|
7ca4a1 |
+ else if (val < 0)
|
|
|
7ca4a1 |
+ {
|
|
|
7ca4a1 |
+ errno=EINVAL;
|
|
|
7ca4a1 |
+ return UINT32_MAX;
|
|
|
7ca4a1 |
+ }
|
|
|
7ca4a1 |
+ /* If errno was set, we'll pass it back as-is */
|
|
|
7ca4a1 |
+ return (uint32_t)val;
|
|
|
7ca4a1 |
}
|
|
|
7ca4a1 |
-#endif /* WANT_STRTOUI */
|
|
|
7ca4a1 |
diff --git a/nslcd/common.h b/nslcd/common.h
|
|
|
7ca4a1 |
index 26fcf48ae2a6dc50bc97fab238ecc9a1879342ce..97d386eaf1f6881182729c5d8e46ce30d2d28eba 100644
|
|
|
7ca4a1 |
--- a/nslcd/common.h
|
|
|
7ca4a1 |
+++ b/nslcd/common.h
|
|
|
7ca4a1 |
@@ -161,31 +161,10 @@ void invalidator_do(enum ldap_map_selector map);
|
|
|
7ca4a1 |
#define BUFLEN_HOSTNAME 256 /* host names or FQDN (and safe version) */
|
|
|
7ca4a1 |
#define BUFLEN_MESSAGE 1024 /* message strings */
|
|
|
7ca4a1 |
|
|
|
7ca4a1 |
-/* provide strtouid() function alias */
|
|
|
7ca4a1 |
-#if SIZEOF_UID_T == SIZEOF_UNSIGNED_LONG_INT
|
|
|
7ca4a1 |
-#define strtouid (uid_t)strtoul
|
|
|
7ca4a1 |
-#elif SIZEOF_UID_T == SIZEOF_UNSIGNED_LONG_LONG_INT
|
|
|
7ca4a1 |
-#define strtouid (uid_t)strtoull
|
|
|
7ca4a1 |
-#elif SIZEOF_UID_T == SIZEOF_UNSIGNED_INT
|
|
|
7ca4a1 |
-#define WANT_STRTOUI 1
|
|
|
7ca4a1 |
-#define strtouid (uid_t)strtoui
|
|
|
7ca4a1 |
-#else
|
|
|
7ca4a1 |
-#error unable to find implementation for strtouid()
|
|
|
7ca4a1 |
-#endif
|
|
|
7ca4a1 |
|
|
|
7ca4a1 |
-/* provide strtogid() function alias */
|
|
|
7ca4a1 |
-#if SIZEOF_GID_T == SIZEOF_UNSIGNED_LONG_INT
|
|
|
7ca4a1 |
-#define strtogid (gid_t)strtoul
|
|
|
7ca4a1 |
-#elif SIZEOF_GID_T == SIZEOF_UNSIGNED_LONG_LONG_INT
|
|
|
7ca4a1 |
-#define strtogid (gid_t)strtoull
|
|
|
7ca4a1 |
-#elif SIZEOF_GID_T == SIZEOF_UNSIGNED_INT
|
|
|
7ca4a1 |
-#ifndef WANT_STRTOUI
|
|
|
7ca4a1 |
-#define WANT_STRTOUI 1
|
|
|
7ca4a1 |
-#endif
|
|
|
7ca4a1 |
-#define strtogid (gid_t)strtoui
|
|
|
7ca4a1 |
-#else
|
|
|
7ca4a1 |
-#error unable to find implementation for strtogid()
|
|
|
7ca4a1 |
-#endif
|
|
|
7ca4a1 |
+uint32_t strtoid(const char *nptr,char **endptr,int base);
|
|
|
7ca4a1 |
+#define strtouid (uid_t)strtoid
|
|
|
7ca4a1 |
+#define strtogid (gid_t)strtoid
|
|
|
7ca4a1 |
|
|
|
7ca4a1 |
#ifdef WANT_STRTOUI
|
|
|
7ca4a1 |
/* provide a strtoui() if it is needed */
|
|
|
7ca4a1 |
--
|
|
|
7ca4a1 |
2.14.3
|
|
|
7ca4a1 |
|