Issue 441 - ldap_chase_referrals in libldap/request.c needs a call to hex_unescape?to
Summary: ldap_chase_referrals in libldap/request.c needs a call to hex_unescape?to
Status: VERIFIED FIXED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: slapd (show other issues)
Version: unspecified
Hardware: All All
: --- normal
Target Milestone: ---
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2000-02-05 00:54 UTC by mei@isi.edu
Modified: 2014-08-01 21:06 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description mei@isi.edu 2000-02-05 00:54:38 UTC 
Full_Name: Mei-Hui Su
Version: 1.2.7,1.2.8, dev
OS: solaris 2.7
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (128.9.64.206)


Hi,

  While chasing referrals in request.c's ldap_chase_referral,
we found the new 'ref' being parsed out and encoded into a new
ber does not work right all the time unless we call 
ldap_pvt_hex_unescape on the 'ref' first. I am not sure if this
is a problem because we often get back a huge list of referrals
from a search request or trigger referrals on top of previous 
referral. 

Thanks

mei
Comment 1 Kurt Zeilenga 2000-03-18 23:01:28 UTC 
Can you provide a patch for review?


At 12:54 AM 2/5/00 GMT, mei@isi.edu wrote:
>Full_Name: Mei-Hui Su
>Version: 1.2.7,1.2.8, dev
>OS: solaris 2.7
>URL: ftp://ftp.openldap.org/incoming/
>Submission from: (NULL) (128.9.64.206)
>
>
>Hi,
>
>  While chasing referrals in request.c's ldap_chase_referral,
>we found the new 'ref' being parsed out and encoded into a new
>ber does not work right all the time unless we call 
>ldap_pvt_hex_unescape on the 'ref' first. I am not sure if this
>is a problem because we often get back a huge list of referrals
>from a search request or trigger referrals on top of previous 
>referral. 
>
>Thanks
>
>mei
>
>
>
>
Comment 2 Kurt Zeilenga 2000-03-20 12:57:15 UTC 
changed notes
changed state Open to Feedback
Comment 3 Kurt Zeilenga 2000-03-20 12:57:24 UTC 
moved from Incoming to Software Bugs
Comment 4 Kurt Zeilenga 2000-03-20 20:49:46 UTC 
I'm thinking that the call should be before the strlen( ref )
not after.

Comments?

At 12:40 PM 3/20/00 -0800, Mei-Hui Su wrote:
>cvs diff request.c | more
>Index: request.c
>===================================================================
>RCS file: /repo/OpenLDAP/pkg/ldap/libraries/libldap/request.c,v
>retrieving revision 1.43
>diff -r1.43 request.c
>630a631,633
>> 
>>                 ldap_pvt_hex_unescape(ref);
>> 
>
>
>mei
>
>
Comment 5 Kurt Zeilenga 2000-09-01 11:50:25 UTC 
changed state Feedback to Closed
Comment 6 OpenLDAP project 2014-08-01 21:06:53 UTC 
Applied to devel.  Needs review.