[SSSD] [PATCHES] Initgroups support for RFC2307bis nested groups (master)
Jakub Hrozek
jhrozek at redhat.com
Mon Oct 18 11:15:52 UTC 2010
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 10/18/2010 03:04 AM, Stephen Gallagher wrote:
> Patch 0001: Modify sysdb_[add|remove]_group_member to accept users and
> groups
>
> Previously, it assumed that all members were users. This changes
> the interface so that either a user or a group can be specified.
>
> Also, it eliminates the need for a memory context to be passed,
> since the internal memory should be self-contained.
>
>
Ack
> Patch 0002: Add proper nested initgroup support for RFC2307bis servers
>
>
> Both of these patches are forward-ported from SSSD 1.2.4 (adjusting for
> sysdb changes).
>
Two minor issues - in rfc2307bis_nested_groups_step(), the second
sysdb_attrs_get_string() for SYSDB_NAME is not needed.
In rfc2307bis_nested_groups_process(), there should be a return
following the tevent_req_error() after calling
rfc2307bis_nested_groups_step(). The same code is also in
rfc2307bis_nested_groups_done().
Otherwise looks OK.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
iEYEARECAAYFAky8LGgACgkQHsardTLnvCVZnACfT+Nt4g+ieJuZSDcddWFeaj5g
ESYAmgN1TpVc98TYmU02GXd4uPn91AX5
=1kP0
-----END PGP SIGNATURE-----
More information about the sssd-devel
mailing list