[PATCH] LDAP EXOP password change

[simo]

On Mon, 2008-11-03 at 10:52 +0200, Michael Gorven wrote:
> On Monday 03 November 2008 10:21:40 Volker Lendecke wrote:
> > "man ber_printf" does not show the "N" as a possible
> > parameter. Has the OpenLDAP API changed?
> 
> I did notice that it isn't listed in the manpage. Here are the relevant parts 
> from liblber/encode.c.
> 
> /* N tag */
> static ber_tag_t lber_int_null = 0;
> 
>         case 'n':   /* null */
>             rc = ber_put_null( ber, ber->ber_tag );
>             break;
> 
>         case 'N':   /* Debug NULL */
>             if( lber_int_null != 0 ) {
>                 /* Insert NULL to ensure peer ignores unknown tags */
>                 rc = ber_put_null( ber, lber_int_null );
>             } else {
>                 rc = 0;
>             }
>             break;
> 
> So it seems that 'N' doesn't add anything to the packet (which is what I saw 
> in the packet traces). I've also tested this patch with Samba 3.0.28a, but 
> with the same version of OpenLDAP.
> 
> Perhaps someone could test this patch to see if it works in a different setup.

I know mozldap libraries do not like 'N'.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Senior Software Engineer at Red Hat Inc. <simo at redhat.com>