diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2010-10-15 15:57:10 +0200 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-10-18 13:12:04 -0400 |
| commit | 27c67307976a60088ca301e07404bdb52740c3af (patch) | |
| tree | 698c9331ad91cd71687055f3cf4951dbc5a755b3 /src/providers/ldap/ldap_id.c | |
| parent | d80a670c94e5a0e21702b8cd8ac5a66fbba81178 (diff) | |
| download | sssd-27c67307976a60088ca301e07404bdb52740c3af.tar.gz sssd-27c67307976a60088ca301e07404bdb52740c3af.tar.xz sssd-27c67307976a60088ca301e07404bdb52740c3af.zip | |
Use unsigned long for conversion to id_t
We used strtol() on a number of places to convert into uid_t or gid_t
from a string representation such as LDAP attribute, but on some
platforms, unsigned long might be necessary to store big id_t values.
This patch converts to using strtoul() instead.
Diffstat (limited to 'src/providers/ldap/ldap_id.c')
| -rw-r--r-- | src/providers/ldap/ldap_id.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/providers/ldap/ldap_id.c b/src/providers/ldap/ldap_id.c index be4642aab..135e37084 100644 --- a/src/providers/ldap/ldap_id.c +++ b/src/providers/ldap/ldap_id.c @@ -220,7 +220,7 @@ static void users_get_done(struct tevent_req *subreq) case BE_FILTER_IDNUM: errno = 0; - uid = (uid_t)strtol(state->name, &endptr, 0); + uid = (uid_t) strtoul(state->name, &endptr, 0); if (errno || *endptr || (state->name == endptr)) { tevent_req_error(req, errno); return; @@ -446,7 +446,7 @@ static void groups_get_done(struct tevent_req *subreq) case BE_FILTER_IDNUM: errno = 0; - gid = (gid_t)strtol(state->name, &endptr, 0); + gid = (gid_t) strtoul(state->name, &endptr, 0); if (errno || *endptr || (state->name == endptr)) { tevent_req_error(req, errno); return; |