[SSSD] [PATCHES] idmap: add API to free allocated SIDs

Sumit Bose sbose at redhat.com
Wed Nov 6 09:16:58 UTC 2013


On Tue, Nov 05, 2013 at 12:29:25PM +0100, Pavel Březina wrote:
> On 11/04/2013 04:39 PM, Pavel Brezina wrote:
> >
> >
> >----- Original Message -----
> >>From: "Sumit Bose" <sbose at redhat.com>
> >>To: sssd-devel at lists.fedorahosted.org
> >>Sent: Monday, November 4, 2013 3:03:04 PM
> >>Subject: Re: [SSSD] [PATCHES] idmap: add API to free allocated SIDs
> >>
> >>On Mon, Nov 04, 2013 at 12:13:52PM +0100, Pavel Březina wrote:
> >>>https://fedorahosted.org/sssd/ticket/2133
> >>
> >
> >Hi,
> >thanks for the review.
> >
> >>Patches are looking good, I wonder why you did not cover binary SIDs,
> >>i.e. *_to_bin_sid() calls?
> >
> >Ha, I knew I forgot something. I'll send another version that will include binary SIDs.
> 
> New patches are attached.
> 

ACK for 1-5. I think Jakub should decide how to handle the updates of
the version-info.

bye,
Sumit



More information about the sssd-devel mailing list