[Freeipa-devel] [PATCH 0/1] add pam support for sssd

[Simo Sorce]

On Tue, 2009-02-24 at 23:15 +0100, Sumit Bose wrote:
> Hi,
> 
> this patch will add pam support with an LDAP backend to sssd. It is
> quite huge and moves a couple of files to new directories. There are
> some rough edges but I think it might be easier to push now and smooth
> them later.

Sumit,
I've adapted the patch to the current code in master and split it into 2
parts, the client and the server code.

I also fixed some minor issues in the server code wrt allocation,
although you still have to add code to free arrays returned by
dbus_message_get_args as arrays are allocated.

I also fixed a bug in the pam protocol where you were using "int" as the
size of a filed. I changed it to int32_t, because int can have different
sized while we must keep the same protocol working even on machines that
support multiple arch libs running at the same time like x86/x86_64
ppc32/ppc64 etc...

(int in this case should not change size IIRC, but it is better to
always make sure to use a specific sized type).

Patches build and the client server pipe protocol works, but I had no
time to test the ldap backend properly as I didn't have an ldap server
handy at the moment.

Attached the client patch, server patch will follow in the next email.

Simo.


-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Add-PAM-client.patch
Type: text/x-patch
Size: 290858 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090224/2a344cf7/attachment.bin>