Wireshark-commits: [Wireshark-commits] master e1de06f: SSH: don't display "Payload" if it's empty
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: Sun, 10 Aug 2014 18:08:12 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e1de06f554ff4ef5566ae386ecbc242e9b87980b
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

e1de06f by Роман Донченко (dpb@xxxxxxxxxxxxxx):

    SSH: don't display "Payload" if it's empty
    
    The "Payload" field only makes sense for packets that have unknown
    type or are unexpectedly long. For normal traces, it will always be
    empty, so hide it.
    
    Note: this length check used to be implicitly performed by
    ssh_proto_tree_add_item before 2aa66aa2b2daf0693e405371a5f9037a43896441,
    so this just restores the former behaviour.
    
    Change-Id: I948935bce660018377a004c661b829a19eb0a53b
    Reviewed-on: https://code.wireshark.org/review/3535
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  ed85bd3   [Automatic manuf, services and enterprise-numbers update for 2014-08-10]
    adds  e1de06f   SSH: don't display "Payload" if it's empty


Summary of changes:
 epan/dissectors/packet-ssh.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)