Wireshark-commits: [Wireshark-commits] master 914bb15: TLS: Limit the number of DNs in Certificate
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, 09 Mar 2020 04:42:38 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=914bb159f781c2f3a788fb68a3bacd3f54e24179
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

914bb15 by Andre Luyer (andre@xxxxxxxx):

    TLS: Limit the number of DNs in Certificate Request messages
    
    When the Distinguished Names list is large in the Certificate Request it can
    trigger the "Dissector bug warning". Having a huge list in the tree pane is
    not useful anyway so this change limits the amount of DNs added to the tree,
    preventing this fault condition.
    It is indicated by adding a "[Tree view truncated]" item.
    
    A side effect is that a handshake is no longer incorrectly flagged as 'resumed'
    because the Server Hello Done in the same packet is now dissected.
    
    Bug: 16202
    Change-Id: Ib315940dcabc2d6b31cf3562354214158ea545a5
    Reviewed-on: https://code.wireshark.org/review/36314
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  839209d   Fix formatting of the start time and elapsed time.
     add  914bb15   TLS: Limit the number of DNs in Certificate Request messages


Summary of changes:
 epan/dissectors/packet-tls-utils.c | 15 ++++++++++++++-
 epan/dissectors/packet-tls-utils.h |  8 +++++++-
 2 files changed, 21 insertions(+), 2 deletions(-)