[SSSD] [PATCH] Save dummy groups to cache during initgroups (master)
Stephen Gallagher
sgallagh at redhat.com
Fri Oct 15 13:04:31 UTC 2010
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 10/15/2010 07:39 AM, Stephen Gallagher wrote:
> On 10/15/2010 07:12 AM, Stephen Gallagher wrote:
>> On 10/15/2010 04:01 AM, Jakub Hrozek wrote:
>>> On 10/14/2010 08:48 PM, Stephen Gallagher wrote:
>>>> Patch 0001: Ack.
>
>
>>>> Patch 0002: Nack
>
>>>> I'd rather you used sysdb_attrs_get_string() to get the SYSDB_NAME in
>>>> sdap_add_incomplete_groups().
>
>
>>> Done.
>
>>>> strtol() is also not safe here, as gid_t is a uint32_t, which on 32-bit
>>>> platforms may require unsigned long.
>
>
>>> Done. But IIRC I copied this bit from elsewhere in the code. Do we want
>>> to have a cleanup ticket to use the new sysdb_attrs_get_ulong() facility?
>
>
>> Please open a ticket. If possible, please try to fix this in time for
>> 1.4.0. I have a feeling we have issues with handling high-value UID/GIDs.
>
>> I'm going to ask QE to test this.
>
>>>> Also, since it would be a very useful thing to have, please add a
>>>> function sysdb_attrs_get_ulong() and use that for getting the GID.
>
>
>>> This is implemented in new patch 0001.
>
>
>
>> Review coming a little later.
>
>
>
> Ack to all three. Good work!
>
>
Pushed to master.
- --
Stephen Gallagher
RHCE 804006346421761
Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
iEYEARECAAYFAky4UV8ACgkQeiVVYja6o6P7hQCeLwkQ8prcJ4vNo7yddtlGKkpA
vksAn3jEbd6gqr23e3T5WAeMtkLHOV8p
=doww
-----END PGP SIGNATURE-----
More information about the sssd-devel
mailing list