Re: special characters in passwords + FR + ldap
Hi,
From: "Natalia Escalera" <nescalera@gmail.com>
We have made a small fix to the ldap-module (as seen in the link to the mailing list archive). I don't know if this has been fixed in 1.1.0. I once had a quick look at the ldap-module of 1.1.0, it should be quite easy to test if it still fails.
The password issue is also in FR 1.1.0.
Thats weird. The bug is so easy to spot and should be trivial to fix. And I think it will affect many FR installations. -- Tero Turtiainen Technology Services Capgemini tero.turtiainen@capgemini.com This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.
Hello, I tried the patch on http://bugs.freeradius.org/showattachment.cgi?attach_id=112 for the function ldap_pairget in the file rlm_ldap.c but it gives me an error when executing the 'make' command saying that 'token' was already declared. This is where token is declared: --------------------------------------------------------------- LRAD_TOKEN token; + LRAD_TOKEN token, operator; --------------------------------------------------------------- May be the file in that page is corrupted. Can you please send me a copy of the file for the patch. Thank you in advance, Natalia. On 3/8/06, Turtiainen, Tero <tero.turtiainen@capgemini.com> wrote:
Hi,
From: "Natalia Escalera" <nescalera@gmail.com>
We have made a small fix to the ldap-module (as seen in the link to the mailing list archive). I don't know if this has been fixed in 1.1.0. I once had a quick look at the ldap-module of 1.1.0, it should be quite easy to test if it still fails.
The password issue is also in FR 1.1.0.
Thats weird. The bug is so easy to spot and should be trivial to fix. And I think it will affect many FR installations. -- Tero Turtiainen Technology Services Capgemini tero.turtiainen@capgemini.com
This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.
- List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
Hi,
I tried the patch on http://bugs.freeradius.org/showattachment.cgi?attach_id=112 for the function ldap_pairget in the file rlm_ldap.c but it gives me an error when executing the 'make' command saying that 'token' was already declared. This is where token is declared:
--------------------------------------------------------------- LRAD_TOKEN token; + LRAD_TOKEN token, operator; ---------------------------------------------------------------
May be the file in that page is corrupted. Can you please send me a copy of the file for the patch.
could you send the rlm_ldap.c file that you now have - your patching may have got hosed alan
Hello, I attached a copy of the file rlm_ldap.c of radius 1.1.0 .Thank you, Natalia On 3/8/06, A.L.M.Buxey@lboro.ac.uk <A.L.M.Buxey@lboro.ac.uk> wrote:
Hi,
I tried the patch on http://bugs.freeradius.org/showattachment.cgi?attach_id=112 for the function ldap_pairget in the file rlm_ldap.c but it gives me an error when executing the 'make' command saying that 'token' was already declared. This is where token is declared:
--------------------------------------------------------------- LRAD_TOKEN token; + LRAD_TOKEN token, operator; ---------------------------------------------------------------
May be the file in that page is corrupted. Can you please send me a copy of the file for the patch.
could you send the rlm_ldap.c file that you now have - your patching may have got hosed
alan - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
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
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@lboro.ac.uk <A.L.M.Buxey@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
Hello, Do you have any suggestion of how to fix the problem? Thanks, Natalia. On 3/9/06, Natalia Escalera <nescalera@gmail.com> wrote:
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@lboro.ac.uk <A.L.M.Buxey@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
Hi, I was wondering if someone has any idea of how to solve the problem of special characters(e.g. $) in FreeRadius 1.1.0. Help is very appreciated. Thank you, Natalia. On 3/10/06, Natalia Escalera <nescalera@gmail.com> wrote:
Hello,
Do you have any suggestion of how to fix the problem?
Thanks, Natalia.
On 3/9/06, Natalia Escalera <nescalera@gmail.com> wrote:
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@lboro.ac.uk <A.L.M.Buxey@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
participants (3)
-
A.L.M.Buxey＠lboro.ac.uk -
Natalia Escalera -
Turtiainen, Tero