deac/ruby-net-ldap
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

noticed a possible parameter problem in paged-search,

Browse source 
added comments.
This commit is contained in:
blackhedd 2006-09-02 14:45:25 +00:00
parent 9ece3ff393
commit 8d8a2be5f8
1 changed files with 8 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/net/ldap.rb
View file

@ -1217,6 +1217,14 @@ module Net
# then query again for the next page of results. # then query again for the next page of results.
# If not, we're done. # If not, we're done.
# Don't screw this up or we'll break every search we do. # Don't screw this up or we'll break every search we do.
#
# Noticed 02Sep06, look at the read_ber call in this loop,
# shouldn't that have a parameter of AsnSyntax? Does this
# just accidentally work? According to RFC-2696, the value
# expected in this position is of type OCTET STRING, covered
# in the default syntax supported by read_ber, so I guess
# we're ok.
#
more_pages = false more_pages = false
if result_code == 0 and controls if result_code == 0 and controls
controls.each do |c| controls.each do |c|