diff options
author | Guy Harris <guy@alum.mit.edu> | 2014-01-21 09:56:34 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2014-01-21 09:56:34 +0000 |
commit | 4d9475e4ef71951d60120746f8bb130d3918f015 (patch) | |
tree | a9de685aba075743c570f46a7bd67a6aa9a8e26e /epan/dissectors/pidl | |
parent | 35d0d0f1ae67528f1f2f7f23651b498178f7143a (diff) |
Get rid of tvb_get_faked_unicode() - tvb_get_string_enc() does the job
better.
We don't need eventlog_get_unicode_string_length() in the eventlog
dissector, either - tvb_unicode_strsize() does the job just as well.
svn path=/trunk/; revision=54874
Diffstat (limited to 'epan/dissectors/pidl')
-rw-r--r-- | epan/dissectors/pidl/eventlog.cnf | 48 |
1 files changed, 12 insertions, 36 deletions
diff --git a/epan/dissectors/pidl/eventlog.cnf b/epan/dissectors/pidl/eventlog.cnf index 8385363078..cc8491795f 100644 --- a/epan/dissectors/pidl/eventlog.cnf +++ b/epan/dissectors/pidl/eventlog.cnf @@ -104,49 +104,27 @@ eventlog_dissect_element_Record_sid_offset(tvbuff_t *tvb, int offset, packet_inf return offset; } - -static int -eventlog_get_unicode_string_length(tvbuff_t *tvb, int offset) -{ - int len; - - len=0; - while(1){ - if(!tvb_get_ntohs(tvb, offset+len*2)){ - len++; - break; - } - len++; - } - - return len; -} - static int eventlog_dissect_element_Record_source_name(tvbuff_t *tvb, int offset, packet_info *pinfo _U_, proto_tree *tree, guint8 *drep _U_) { - char *str; - int len; + guint len; - len=eventlog_get_unicode_string_length(tvb, offset); - str=tvb_get_faked_unicode(wmem_packet_scope(), tvb, offset, len, TRUE); - proto_tree_add_string_format(tree, hf_eventlog_Record_source_name, tvb, offset, len*2, str, "source_name: %s", str); + len=tvb_unicode_strsize(tvb, offset); + proto_tree_add_item(tree, hf_eventlog_Record_source_name, tvb, offset, len, ENC_UTF_16|ENC_LITTLE_ENDIAN); - offset+=len*2; + offset+=len; return offset; } static int eventlog_dissect_element_Record_computer_name(tvbuff_t *tvb, int offset, packet_info *pinfo _U_, proto_tree *tree, guint8 *drep _U_) { - char *str; - int len; + guint len; - len=eventlog_get_unicode_string_length(tvb, offset); - str=tvb_get_faked_unicode(wmem_packet_scope(), tvb, offset, len, TRUE); - proto_tree_add_string_format(tree, hf_eventlog_Record_computer_name, tvb, offset, len*2, str, "computer_name: %s", str); + len=tvb_unicode_strsize(tvb, offset); + proto_tree_add_item(tree, hf_eventlog_Record_computer_name, tvb, offset, len, ENC_UTF_16|ENC_LITTLE_ENDIAN); - offset+=len*2; + offset+=len; return offset; } @@ -176,13 +154,11 @@ static int eventlog_dissect_element_Record_strings(tvbuff_t *tvb, int offset, packet_info *pinfo _U_, proto_tree *tree, guint8 *drep _U_) { while(string_offset && num_of_strings){ - char *str; - int len; + guint len; - len=eventlog_get_unicode_string_length(tvb, string_offset); - str=tvb_get_faked_unicode(wmem_packet_scope(), tvb, string_offset, len, TRUE); - proto_tree_add_string_format(tree, hf_eventlog_Record_string, tvb, string_offset, len*2, str, "string: %s", str); - string_offset+=len*2; + len=tvb_unicode_strsize(tvb, string_offset); + proto_tree_add_item(tree, hf_eventlog_Record_string, tvb, string_offset, len, ENC_UTF_16|ENC_LITTLE_ENDIAN); + string_offset+=len; num_of_strings--; } |