Wireshark-commits: [Wireshark-commits] master 54261cc: SMB2: remove duplicate session info
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: Fri, 28 Feb 2020 14:54:22 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=54261ccaf231051f2024f5523a51640d9ce529b0
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

54261cc by Aurelien Aptel (aaptel@xxxxxxxx):

    SMB2: remove duplicate session info
    
    When smb2_session_info() was factored out a bug was introduced where
    instead of appending text in the parent it appended text in the new
    sub item, showing it twice. Fix that by appending to the parent item
    instead.
    
    Change-Id: I87c681cd5ea233b3a8c451d0dde2cd2ddf8da48e
    Reviewed-on: https://code.wireshark.org/review/36191
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  f222e4b   pfcp: update IE 8.2.12  Transport Level Marking
     add  54261cc   SMB2: remove duplicate session info


Summary of changes:
 epan/dissectors/packet-smb2.c | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)