Wireshark-commits: [Wireshark-commits] master-2.0 c4d5050: USB: Fix bytes highlighting on setup par
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 28 Oct 2015 11:44:37 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c4d5050cbc9dd5a26319e49fa7f4d26575c7f190
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

c4d5050 by Michal Labedzki (michal.labedzki@xxxxxxxxx):

    USB: Fix bytes highlighting on setup part
    
    In a case that Setup part is used as payload within rest of data
    (setup and remaining data merged), no bytes are highlighted
    on bytes pane. Also move next dissector tree under root tree.
    
    Change-Id: If127f6f2061c60795f2b9940c3a6cb6034cdbdf7
    Reviewed-on: https://code.wireshark.org/review/11026
    Petri-Dish: Michal Labedzki <michal.labedzki@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 5e8c6291f4f3e4f237eed5a92afb5d320c66544d)
    Reviewed-on: https://code.wireshark.org/review/11325
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  61b634a   Bluetooth: ATT: Add missing "Generic" items for Appearance
    adds  c4d5050   USB: Fix bytes highlighting on setup part


Summary of changes:
 epan/dissectors/packet-usb.c |   26 ++++++++++++++++++++------
 1 file changed, 20 insertions(+), 6 deletions(-)