On Tue, Oct 19, 2010 at 04:28:09PM -0400, Simo Sorce wrote: > On Tue, 19 Oct 2010 16:06:22 -0400 > Stephen Gallagher <sgall...@redhat.com> wrote: > > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA1 > > > > On 10/18/2010 03:16 PM, Jakub Hrozek wrote: > > > 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. > > > > > > It has been discovered that LDAP only supports signed 32-bit integers > > for the UID, so there is no need for this patch. No sense in > > backporting it, in that case. > > This doesn't sound right. > > >From RFC 4517: > > 3.3.16. Integer > > A value of the Integer syntax is a whole number of unlimited > magnitude. [..] > > > Simo.
I agree and additionally, ldap_user_uid_number can be used to read the UID from any other attribute where larger numbers are supported. bye, Sumit > > -- > Simo Sorce * Red Hat, Inc * New York > _______________________________________________ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://fedorahosted.org/mailman/listinfo/sssd-devel _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel