[SSSD] [PATCH] free sid obtained from sss_idmap_unix_to_sid()

Stephen Gallagher sgallagh at redhat.com
Wed Oct 30 15:26:21 UTC 2013


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 10/30/2013 10:29 AM, Pavel Březina wrote:
> 


Nack. This happens to work by coincidence (we're initializing the
idmap with a talloc allocator), but the real problem here is that we
don't have a proper free() function to deal with sss_idmap_unix_to_*()
functions (and friends). We allow the sss_idmap_init() to take a
custom allocator, which means we need a custom free as well. The only
one we offer right now is for freeing the entire idmap_ctx.

Please add an appropriate sss_idmap_free_data() function and use that.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.15 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlJxJR0ACgkQeiVVYja6o6OcBACgiaVM8Lj/L7TzuPPU1skZ7eQB
yrAAnAlAhEIZFzymOMvsuIpG36kP4cUs
=MlLW
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list