Wireshark-commits: [Wireshark-commits] master 98b7f21: [Reassembly] Fix a reassembly case where the

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 18 Mar 2015 05:32:12 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=98b7f213706c9b4d0636d49d635fa1443a5748c4
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

98b7f21 by AndersBroman (anders.broman@xxxxxxxxxxxx):

    [Reassembly] Fix a reassembly case where the two fragments are in the same
    frame
    but in different SCTP DATA chunks, whitout the patch the message is
    reassembled in both chunks leading to duplicated upper layer PDU:s in the
    frame.
    
    Change-Id: Ie31142c38c728018178947544b571622447d8e8f
    Reviewed-on: https://code.wireshark.org/review/7716
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  ae2d244   Remove proto_tree_add_text from packet-bootp.c
    adds  98b7f21   [Reassembly] Fix a reassembly case where the two fragments are in the same frame but in different SCTP DATA chunks, whitout the patch the message is reassembled in both chunks leading to duplicated upper layer PDU:s in the frame.


Summary of changes:
 epan/reassemble.c |    9 ++++++++-
 epan/reassemble.h |   41 ++++++++++++++++++++++-------------------
 2 files changed, 30 insertions(+), 20 deletions(-)