diff options
author | Guy Harris <guy@alum.mit.edu> | 2019-07-17 07:26:54 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2019-07-17 15:07:18 +0000 |
commit | 0696d9349524ae06d17e774f5590a0b94ebae1c9 (patch) | |
tree | 466a7bf02be2a9840db0bc28ca4d4cf6fb31672f /.gitignore | |
parent | f735f916a9797838c497d2c2b535f2518f8187db (diff) |
Don't just grab raw string data with tvb_memcpy().
Use proto_tree_add_item_ret_display_string() routines to add strings if
we want to display the string's value in a column, and just use
proto_tree_add_item() if we don't need the string's value. That way,
all strings are fetched using an encoding value, to properly map to
UTF-8, and, if necessary, are formatted for display.
Add comments about fields that have type VisibleString, asking whether
that means "ASCII" or "just the "Basic" part of ISO 646", where the
latter is ENC_ISO_646_BASIC.
Add a comment about fields using "The definition of IETF RFC 5890" -
that means "Punycode", but I don't think we map Punycode to Unicode;
perhaps we should. Also ask whether that also implies RFC 1035's
encoding of domain names as sequences of counted strings.
Change-Id: Ie8d5bc18d2846f8d723019b22f69a50ce55f9a9b
Reviewed-on: https://code.wireshark.org/review/33986
Petri-Dish: Guy Harris <guy@alum.mit.edu>
Tested-by: Petri Dish Buildbot
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to '.gitignore')
0 files changed, 0 insertions, 0 deletions