Wireshark-commits: [Wireshark-commits] master 2263ce4: TLS: fix reassembly regression when extendin
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: Tue, 02 Oct 2018 04:43:17 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2263ce4f63f91f58d840c628dccaec6a72a0d6fc
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

2263ce4 by Peter Wu (peter@xxxxxxxxxxxxx):

    TLS: fix reassembly regression when extending a PDU
    
    The fragment identifier was changed, but fragment_set_partial_reassembly
    was left unchanged which broke reassembly/decryption when a subdissector
    finishes reassembly, but then requests further reassembly (like HTTP
    with chunked encoding). This bug results in a faulty "Reassembly error,
    protocol TLS: New fragment past old data limits" message.
    
    Bug: 15158
    Change-Id: I636ef623dd65c3feb58da29adf7125d8458f038f
    Fixes: v2.9.0rc0-2003-gbe16d87c60 ("TLS: fix broken reassembly with multiple PDUs in the same frame")
    Reviewed-on: https://code.wireshark.org/review/29967
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  fdecc62   CMake: Add a systemd RPM check.
     add  2263ce4   TLS: fix reassembly regression when extending a PDU


Summary of changes:
 epan/dissectors/packet-tls.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)