Wireshark-bugs: [Wireshark-bugs] [Bug 5206] Wireshark incorrectly processes SMPP optional parame
Date: Sun, 24 Nov 2013 15:14:12 +0000

Comment # 4 on bug 5206 from
Created attachment 12163 [details]
Experimental patch (not for commit)

With the attached patch, a new procedure "smpp_handle_tlv_string_z" which
validates the actual length of the string against that reported by the Length
field of the TLV. 

If there is a mismatch, then expert information is added. 

A new preference is also introduced which allows dissection to be done using
the reported/claimed length of the TLVs instead of the determined length
(determined using the null terminator).

Also attaching screehshots of how the changes look. 

If this is an acceptable approach, then I will apply the changes to all
relevant TLVs. At the moment, it is only applied for the reported TLV which was
wrongly encoded.


You are receiving this mail because:
  • You are the assignee for the bug.
  • You are watching all bug changes.