Wireshark-commits: [Wireshark-commits] master-2.0 96f9bbc: SEL Protocol - Fix 'Fast Meter Config' s
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=96f9bbc5c2353902c574f68e9818570c2002f549
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark
Commits:
96f9bbc by Chris Bontje (cbontje@xxxxxxxxx):
SEL Protocol - Fix 'Fast Meter Config' sub-tree addition typo & 'if (tree)' usage
- The proto_tree_add_subtree call was previously using the length of the full tvb,
instead of len-2 like all other calls. This typo did not previously cause an issue,
but in newer versions of Wireshark it causes an assert.
- When I created the selfm dissector 4-5 years ago, I used packet-synphasor.c
as a template and there was an 'if (tree)' wrapper around several initial GUI calls
including col_*** info column clear/set functions. It looks like in recent years
this layout has been removed (for example, https://code.wireshark.org/review/#/c/6725/ ).
I'm not sure of the exact reasoning behind this, but in Wireshark 2.x (QT), there are
several conditions where in the SEL protocol dissector the info column will be
cleared and not repopulate - I can only assume because col_clear has been called
with no col_add's. Removal of this 'if' condition does correct the issue with no immediate ill effects..
Change-Id: I4c619cb320cd2152e1d4d1b76c809b1498dfda61
Reviewed-on: https://code.wireshark.org/review/17147
Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
(cherry picked from commit ea024e245e3db5f65a99479be54414ba5f2413fc)
Reviewed-on: https://code.wireshark.org/review/17153
Actions performed:
from 2024fd0 DCT2000: fix a buffer overflow in case of malformed packet
adds 96f9bbc SEL Protocol - Fix 'Fast Meter Config' sub-tree addition typo & 'if (tree)' usage
Summary of changes:
epan/dissectors/packet-selfm.c | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
