Wireshark-commits: [Wireshark-commits] master ea024e2: SEL Protocol - Fix 'Fast Meter Config' sub-t

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 18 Aug 2016 20:46:26 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ea024e245e3db5f65a99479be54414ba5f2413fc
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

ea024e2 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>
    

Actions performed:

    from  fce1871   QUIC: Add detection of server port
    adds  ea024e2   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(-)