diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2012-02-29 10:37:41 -0500 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-02-29 13:52:08 -0500 |
| commit | 55fcd37c086f699db1aeac54a3df046b60755ff6 (patch) | |
| tree | 6298b25014fd692b88a50216844dc62d1bfe1c1b /src/util | |
| parent | b7b50b21d2254a079b1b1c299909483d23db1512 (diff) | |
| download | sssd_unused-55fcd37c086f699db1aeac54a3df046b60755ff6.tar.gz sssd_unused-55fcd37c086f699db1aeac54a3df046b60755ff6.tar.xz sssd_unused-55fcd37c086f699db1aeac54a3df046b60755ff6.zip | |
Handle cases where UID is -1
Also removes an unnecessary range check (since it's already handled
by strtoint32()
https://fedorahosted.org/sssd/ticket/1216
Diffstat (limited to 'src/util')
| -rw-r--r-- | src/util/find_uid.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/util/find_uid.c b/src/util/find_uid.c index f4156ce7..77b9f22a 100644 --- a/src/util/find_uid.c +++ b/src/util/find_uid.c @@ -137,7 +137,7 @@ static errno_t get_uid_from_pid(const pid_t pid, uid_t *uid) } else { *e = '\0'; } - num = strtouint32(p, &endptr, 10); + num = (uint32_t) strtoint32(p, &endptr, 10); error = errno; if (error != 0) { DEBUG(1, ("strtol failed [%s].\n", strerror(error))); @@ -148,11 +148,6 @@ static errno_t get_uid_from_pid(const pid_t pid, uid_t *uid) return EINVAL; } - if (num >= UINT32_MAX) { - DEBUG(1, ("uid out of range.\n")); - return ERANGE; - } - } else { DEBUG(1, ("format error\n")); return EINVAL; |