Wireshark-commits: [Wireshark-commits] master ab1a7cc: TLS: fix DISSECTOR_ASSERT for zero-length re
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, 15 May 2019 11:27:19 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ab1a7cc4a5dc5435df50cc480d1f0ae8ad06ee65
Submitter: "Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

ab1a7cc by Peter Wu (peter@xxxxxxxxxxxxx):

    TLS: fix DISSECTOR_ASSERT for zero-length records fragments
    
    When decrypt_ssl3_record is called with a record length of zero, it will
    pass NULL to ssl_data_set because tvb_get_ptr(..., 0) yields NULL. That
    triggers a DISSECTOR_ASSERT. Fix this and add expert info while at it.
    
    Bug: 15780
    Change-Id: I727b511aa48b6e1aeb20a441d1eb9d3627a74413
    Reviewed-on: https://code.wireshark.org/review/33203
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  11110ae   rawshark: fix parameter 'pd' not found in the function declaration [-Wdocumentation]
     add  ab1a7cc   TLS: fix DISSECTOR_ASSERT for zero-length records fragments


Summary of changes:
 epan/dissectors/packet-dtls.c      |  2 +-
 epan/dissectors/packet-tls-utils.c | 17 +++++++++++++++++
 epan/dissectors/packet-tls-utils.h |  3 ++-
 epan/dissectors/packet-tls.c       |  9 ++++++---
 4 files changed, 26 insertions(+), 5 deletions(-)