Comment # 8
on bug 8112
from Martin Kaiser
one more thing: the code in question is
tool_version = tvb_get_ephemeral_unicode_string(tvb, offset,
tool_version_length*2,
ENC_LITTLE_ENDIAN);
proto_tree_add_string(tree, hf_msmms_command_tool_version, tvb,
offset, tool_version_length*2,
format_text((guchar*)tool_version, tool_version_length));
tool_version is a UTF-8 string, i.e. a character can be encoded in one or more
bytes.
Looking at the code of format_text(), I can't see that this handles UTF-8
strings.
Should these two statements be replaced with
proto_tree_add_unicode_string(..., tvb_get_ephemeral_unicode_string(...)) ?
(there's a number of similar code parts)
You are receiving this mail because:
- You are watching all bug changes.
