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
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 > > > >
changed notes changed state Open to Feedback
moved from Incoming to Software Bugs
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 > >
changed state Feedback to Closed
Applied to devel. Needs review.