aboutsummaryrefslogtreecommitdiffstats
path: root/ConfigureChecks.cmake
diff options
context:
space:
mode:
authorcbontje <cbontje@gmail.com>2015-08-07 11:18:00 -0600
committerAnders Broman <a.broman58@gmail.com>2015-11-06 07:52:09 +0000
commit35a13838146ac5d83b57435ff34f2960ea911248 (patch)
treeb78cc34febd561fabd5674fd9fb86c40517ce71c /ConfigureChecks.cmake
parent018e93a384c5aca7471d7d9303af637e71320ea2 (diff)
downloadwireshark-35a13838146ac5d83b57435ff34f2960ea911248.tar.gz
wireshark-35a13838146ac5d83b57435ff34f2960ea911248.tar.bz2
wireshark-35a13838146ac5d83b57435ff34f2960ea911248.zip
Major Re-factoring of the dissector and some new features.
A change-list is as follows: - Removed un-necessary dissector revision updates from the file header since GIT tracks these nicely. - Added proper size detection of Modbus RTU messages (including exception responses), when dealing with partial TCP segment reassembly. - Moved the 'register' decode preferences to the Modbus dissector as TCP vs. RTU granularity isn't needed in this case. - Obsoleted un-unused 'address type format' user preferences - Cleaned up dissect_modbus_data to remove proto_tree_set_text instances. - For decoded register tree objects, use register 'address' instead of 'value' for the filter field to provide a more useful filter. - Added in conversation support, to attempt to track responses back to matching requests. - Use conversation support to attempt to populate proper register address offsets in the response messages. Currently each request is saved and each response looks for the last prior request that matches the function code. - Re-factored Modbus dissector to split apart request vs. response decoding. This has led to cleaner code paths, but some duplication where replies and requests are identical format. Change-Id: I0c86ae85b8ae4cc59b037e5f68f408833205fadd Reviewed-on: https://code.wireshark.org/review/9914 Reviewed-by: Michael Mann <mmann78@netscape.net> Petri-Dish: Michael Mann <mmann78@netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org> Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'ConfigureChecks.cmake')
0 files changed, 0 insertions, 0 deletions