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

2013-12-19 Thread Jakub Hrozek
t; > > > > > >- Original Message - > > > >>From: "Sumit Bose" > > > >>To: sssd-devel@lists.fedorahosted.org > > > >>Sent: Monday, November 4, 2013 3:03:04 PM > > > >>Subject: Re: [SSSD] [PATCHES] idmap: a

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

2013-11-07 Thread Jakub Hrozek
04:39 PM, Pavel Brezina wrote: > >>>> > >>>> > >>>>- Original Message - > >>>>>From: "Sumit Bose" > >>>>>To: sssd-devel@lists.fedorahosted.org > >>>>>Sent: Monday, November 4, 2013 3:03:04 P

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

2013-11-07 Thread Pavel Březina
sssd-devel@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

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

2013-11-07 Thread Jakub Hrozek
it Bose" > > >>To: sssd-devel@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,

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

2013-11-07 Thread Jakub Hrozek
it Bose" > > >>To: sssd-devel@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,

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

2013-11-06 Thread Sumit Bose
vember 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, > &g

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

2013-11-05 Thread Pavel Březina
On 11/04/2013 04:39 PM, Pavel Brezina wrote: - Original Message - From: "Sumit Bose" To: sssd-devel@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 +0

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

2013-11-04 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/04/2013 09:47 AM, Lukas Slebodnik wrote: > On (04/11/13 15:40), Jakub Hrozek wrote: >> On Mon, Nov 04, 2013 at 03:03:04PM +0100, Sumit Bose wrote: >>> On Mon, Nov 04, 2013 at 12:13:52PM +0100, Pavel Březina wrote: https://fedorahosted.org/ss

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

2013-11-04 Thread Pavel Brezina
- Original Message - > From: "Sumit Bose" > To: sssd-devel@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, Pa

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

2013-11-04 Thread Lukas Slebodnik
On (04/11/13 15:40), Jakub Hrozek wrote: >On Mon, Nov 04, 2013 at 03:03:04PM +0100, Sumit Bose wrote: >> On Mon, Nov 04, 2013 at 12:13:52PM +0100, Pavel Březina wrote: >> > https://fedorahosted.org/sssd/ticket/2133 >> >> Patches are looking good, I wonder why you did not cover binary SIDs, >> i.e.

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

2013-11-04 Thread Jakub Hrozek
On Mon, Nov 04, 2013 at 03:03:04PM +0100, Sumit Bose wrote: > On Mon, Nov 04, 2013 at 12:13:52PM +0100, Pavel Březina wrote: > > https://fedorahosted.org/sssd/ticket/2133 > > Patches are looking good, I wonder why you did not cover binary SIDs, > i.e. *_to_bin_sid() calls? > > bye, > Sumit I wou

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

2013-11-04 Thread Sumit Bose
On Mon, Nov 04, 2013 at 12:13:52PM +0100, Pavel Březina wrote: > https://fedorahosted.org/sssd/ticket/2133 Patches are looking good, I wonder why you did not cover binary SIDs, i.e. *_to_bin_sid() calls? bye, Sumit ___ sssd-devel mailing list sssd-devel

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

2013-11-04 Thread Pavel Březina
https://fedorahosted.org/sssd/ticket/2133 From 683b8be4f4d54dc9efc2b35de9d993be0dd69c39 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= Date: Fri, 1 Nov 2013 12:23:23 +0100 Subject: [PATCH 1/5] idmap: add API to free allocated SIDs --- src/lib/idmap/sss_idmap.c | 30