Wireshark-commits: [Wireshark-commits] master 9140921: DOCSIS: Remove concatenated PDU dissection.
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, 20 Feb 2018 06:20:03 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=91409213ad111fb1607636dcb0b69dac9fb2d0c5
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

9140921 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    DOCSIS: Remove concatenated PDU dissection.
    
    The current concatenation PDU support has had serious, repeated problems
    over the years:
    
    fb1ef7b8da
    f6d48e45c8
    3e1828e351
    26a6881014
    625bab309d
    
    Remove it and add a comment recommending iteration.
    
    Bug: 14446
    Change-Id: I947ff8e40e18c4628c9df9233b72dd7776e8233d
    Reviewed-on: https://code.wireshark.org/review/25905
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  8a173c9   Clean up the way we handle the server recv->send duration.
    adds  9140921   DOCSIS: Remove concatenated PDU dissection.


Summary of changes:
 epan/dissectors/packet-docsis.c | 87 +++++++++--------------------------------
 1 file changed, 19 insertions(+), 68 deletions(-)