Wireshark-commits: [Wireshark-commits] master 1573d4d: SCTP: assure size before attempting checksum
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 21:45:21 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1573d4ddb2d0ff4dfa5b41f91f5620f1c7dd475f
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

1573d4d by Jaap Keuter (jaap.keuter@xxxxxxxxx):

    SCTP: assure size before attempting checksum calculation
    
    When checksum calculations are performed (irrespective of type) the
    calculation is assumed to be possible on at least a common header and
    optionally a payload. This assumption was not checked, which could
    lead to out of bound access of packet buffer data.
    Simply adding the assurance that enough buffer data is available avoids
    this out of bound access.
    
    CID 1439698
    
    Change-Id: I5fec69b96b1064ffdda11f51b882fe5775844475
    Reviewed-on: https://code.wireshark.org/review/33185
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  80920b0   Diameter: Fix copy-paste-o
     add  1573d4d   SCTP: assure size before attempting checksum calculation


Summary of changes:
 epan/dissectors/packet-sctp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)