[SSSD] [PATCH] Allow different SID representations in libidmap

Stephen Gallagher sgallagh at redhat.com
Fri Apr 27 17:49:53 UTC 2012


On Fri, 2012-04-27 at 12:54 +0200, Jakub Hrozek wrote:
> On Thu, Apr 26, 2012 at 12:41:58PM +0200, Sumit Bose wrote:
> > Hi,
> > 
> > this patch extends the functionality of libidmap by allow more kind of
> > SID representations, especially the binary format which is used e.g. to
> > store SIDs in AD.
> > 
> > I have added tests for all new functions, but mostly they only test the
> > positve case and do not try to break anything with invalid inputs. Fell
> > free to add more :-)
> > 
> > bye,
> > Sumit
> 
> The code looks good to me, test pass and all the functions are covered
> in the test suite.
> 
> Ack, provided that the interface works for Stephen.

Works fine for me. I've tested it with my patches (now on the list as
"[PATCHES] Add support for ID-mapping with Active Directory domains").

Ack.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120427/d49cd4fa/attachment.sig>


More information about the sssd-devel mailing list