URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0a9e92d585a3b8a42aff8ad968710ea1d6a4e7ff
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark
Commits:
0a9e92d by Guy Harris (guy@xxxxxxxxxxxx):
Use proto_tree_add_expert{_format}() to report errors.
Don't create fake string items.
Have the item cover the part of the packet that has the issue, e.g. if
there's an unexpected identifier, or an invalid length, cover that
rather than the value; that 1) shows where the problem is and 2) avoids
throwing an exception.
Clean up the error messages and expert info items while we're at it.
Change-Id: If698db98158f7a5532a865be02f1028d92af262f
Reviewed-on: https://code.wireshark.org/review/28093
Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
Actions performed:
from 0d25e3d ua3g: updated freeseating message decoding to support ipv6 address in maincpu0 and maincpu1 parameters.
adds 0a9e92d Use proto_tree_add_expert{_format}() to report errors.
Summary of changes:
epan/dissectors/packet-ber.c | 450 +++++++++++++++++++++++++------------------
1 file changed, 259 insertions(+), 191 deletions(-)