Mailing List Archive

rev 18089: /trunk/epan/dissectors/: packet-ldap.c /trunk/asn1/ldap/: ldap.cnf
User: sahlberg
Date: 2006/05/04 05:19 AM

Log:
force the AssertionValue type to be displayed as a STRING instead of as hex BYTES.


Most of the time AssertionValue will contain an ascii string so make it always display as a string to make the display "correct" most of the time insterad of being "wrong" most of the time.

There are situations when AssertionValue contains binary data though and in those cases the display will be "wrong" (but not more wrong than the old dissector anyway)


What someone really should do (someone interested in ldap that is) to make it more correct would be to implement a dissector for AssertionValue in the template file and having the dissector check if any of the bytes of the octet string has a value <32 and if so display it in hex as 0x.....
It all bytes have values >=32 then it shoudlk display it as a string "...
instead.

Someone interested in ldap may spend time on this refinement.

Directory: /trunk/epan/dissectors/
Changes Path Action
+17 -17 packet-ldap.c Modified

Directory: /trunk/asn1/ldap/
Changes Path Action
+1 -0 ldap.cnf Modified



http://anonsvn.ethereal.com/viewcvs/viewcvs.py?rev=18089&view=rev
_______________________________________________
Ethereal-cvs mailing list
Ethereal-cvs@ethereal.com
http://www.ethereal.com/mailman/listinfo/ethereal-cvs