special characters in passwords + FR + ldap

[Natalia Escalera]

Hello,

> how did you patch?

What I did is that I took the rlm_ldap.c from FR 1.1.0 and replaced
the content of the function ldap_pairget with the code shown on
http://bugs.freeradius.org/showattachment.cgi?attach_id=112. Then I
execute the './configure' and 'make' commands

Natalia.

On 3/9/06, A.L.M.Buxey at lboro.ac.uk <A.L.M.Buxey at lboro.ac.uk> wrote:
> Hi,
>
> > I attached a copy of the file rlm_ldap.c of radius 1.1.0
>
> yes, your patched version is clearly borked - as you can see from
> this snippet.
>
> >       int             vals_count;
> >       int             vals_idx;
> >       char           *ptr;
> >       char           *value;
> >       TLDAP_RADIUS   *element;
> >       LRAD_TOKEN      token;
> >       LRAD_TOKEN      token, operator;
> >       int             is_generic_attribute;
> >       char            value[256];
> >       char            buf[MAX_STRING_LEN];
> >       VALUE_PAIR     *pairlist = NULL;
> >       VALUE_PAIR     *newpair = NULL;
> >       char            do_xlat = FALSE;
>
> LRAD_TOKEN has dual definitions. it should look similar to:
>
>        char          **vals;
>        int             vals_count;
>        int             vals_idx;
>        char           *ptr;
>        char           *value;
>        TLDAP_RADIUS   *element;
>        LRAD_TOKEN      token, operator;
>        int             is_generic_attribute;
>        char            buf[MAX_STRING_LEN];
>        VALUE_PAIR     *pairlist = NULL;
>        VALUE_PAIR     *newpair = NULL;
>        char            do_xlat = FALSE;
>
> how did you patch? I notice that the patch is no longer clean against the
> 1.1.x CVS code...which means that more headaches will occur. someone with
> the drive/desire needs to modify the patch for the more recent source
>
> alan
> -
> List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
>