[Date Prev][Date Next] [Chronological] [Thread] [Top]

RE: liblber/io.c:ber_get_next



> -----Original Message-----
> From: Kurt D. Zeilenga [mailto:Kurt@OpenLDAP.org]

> At 12:53 PM 2002-01-10, Howard Chu wrote:
> >The code to deal with multi-byte tags in ber_get_next is byte-order 
> dependent, it will only work on a Big-Endian platform. It seems to 
> me that this code shouldn't even be here since the only valid tag at 
> this point is 0x30. Any opinions?
 
> For the LDAPMessage SEQUENCE, the 0x30 is to be expected.
> However, once could use ber_get_next to read BER/DER
> element which had a multi-octet tag.  If the code could
> can be easily patched to handle correctly and efficiently
> handle multi-byte tags, that is what I would prefer.
> Otherwise, we might as well optimize for single-octet
> tags.

Ok. The code I just checked in is now byte-order independent for multi-octet
tags. It also is about 58% faster overall. (Trivial, ber_get_next was never
a significant amount of time. Dropped from 66th most CPU to 77th.)

  -- Howard Chu
  Chief Architect, Symas Corp.       Director, Highland Sun
  http://www.symas.com               http://highlandsun.com/hyc  
  Symas: Premier OpenSource Development and Support