Wireshark-commits: [Wireshark-commits] master-2.2 0de7372: ssl: Fix Info column separator
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: Mon, 3 Apr 2017 05:34:14 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0de7372ce08b4fd2f2dfa5d2fa57736b2d0758f8
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

0de7372 by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    ssl: Fix Info column separator
    
    Always prefix Info column entries with comma if the column is
    not empty.
    
    Bug: 13539
    Change-Id: I47b43841b092671fe5a74c99ec8aaa15f144bced
    Reviewed-on: https://code.wireshark.org/review/20844
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    (cherry picked from commit 235ee14d2890454f0dd5eda78096fefe1731c0b4)
    Reviewed-on: https://code.wireshark.org/review/20858
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  a2508da   DNS: add an expert info if buffer is too short to compute a key id
    adds  0de7372   ssl: Fix Info column separator


Summary of changes:
 epan/dissectors/packet-ssl.c |   92 +++++++++---------------------------------
 1 file changed, 20 insertions(+), 72 deletions(-)