On Wed, 2012-02-29 at 12:57 -0500, Stephen Gallagher wrote:
On Wed, 2012-02-29 at 17:28 +0100, Pavel Březina wrote:
> Dne 29.2.2012 16:39, Stephen Gallagher napsal(a):
> > Also removes an unnecessary range check (since it's already handled
> > by strtoint32()
> >
> > Fixes:
https://fedorahosted.org/sssd/ticket/1216
>
> ACK
>
> Although would it make sense to do this check inside strtouint32() and
> set errno to EINVAL if the string does not contain unsigned int?
That was the original bug :)
-1 was a valid (according to the kernel) UID, but we were incorrectly
denying it. We need to allow negative values to be specified.
Pushed to master and sssd-1-8.