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

[Jakub Hrozek]

On Thu, Nov 07, 2013 at 10:59:04AM +0100, Jakub Hrozek wrote:
>  On Wed, Nov 06, 2013 at 10:16:58AM +0100, Sumit Bose wrote:
>  > 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(a)redhat.com&gt;
>  > > >>To: sssd-devel(a)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
>  
>  I filed https://fedorahosted.org/sssd/ticket/2145 as Stephen suggested
>  to track pushing the last patch. 
The patch to fix to bump the version info was pushed to master and
sssd-1-11