[SSSD] [PATCH] sss_idmap: add support for samba struct dom_sid

Sumit Bose sbose at redhat.com
Mon Jun 4 12:49:48 UTC 2012


On Mon, Jun 04, 2012 at 02:15:37PM +0200, Jakub Hrozek wrote:
> On Fri, Jun 01, 2012 at 01:06:11PM +0200, Sumit Bose wrote:
> > On Thu, May 31, 2012 at 09:09:53AM +0200, Jakub Hrozek wrote:
> > > On Thu, May 24, 2012 at 03:04:36PM +0200, Sumit Bose wrote:
> > > > Hi,
> > > > 
> > > > this patch allows us besides other conversions to convert the dom_sid
> > > > structure used by samba to strings and back. This structure is used by
> > > > various samba libraries, but there are no public inferfaces for the
> > > > conversion. I've seen Simo adding code to the IPA kdb plugin doing these
> > > > conversions and I need them for the PAC responder as well. So I thought
> > > > it might be useful to put it in a library.
> > > > 
> > > > bye,
> > > > Sumit
> > > 
> > > Hi, the patch no longer applies to configure.ac, can you rebase?
> > 
> > sure, new version attached.
> > 
> > bye,
> > Sumit
> 
> The code looks good to me and the unit tests pass. 
> 
> But the patch adds a new build-time dependency that should be added into
> contrib/sssd.spec.in. I also wonder if there should be a configure-time

ah, I have this in the PAC responder patch. I will extract it there and
add it here.

> switch to compile this feature in or out, mainly because nothing is
> using the feature at the moment?

When starting the patch I was thinking the same, but then I realized
that this would change the API/ABI of the library. So the samba header
files are now required at compile time, but not at run time.

bye,
Sumit

> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://fedorahosted.org/mailman/listinfo/sssd-devel



More information about the sssd-devel mailing list