Wireshark-commits: [Wireshark-commits] master dc81822: Fix size information for sctp.sack_duplicate

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: Sat, 8 Apr 2017 12:29:51 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=dc81822154a24c6eed5095d73319cbee18241a94
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

dc81822 by Marcelo Ricardo Leitner (marcelo.leitner@xxxxxxxxx):

    Fix size information for sctp.sack_duplicate_tsn
    
    It was listed as UINT16 while it's actually UINT32.
    The loop that actually parses the duplicate_tsn's in dissect_sack_chunk
    is already parsing it using 4-bytes, as per SACK_CHUNK_DUP_TSN_LENGTH
    definition.
    
    Change-Id: I90fa115844a397f143d39770da9f5bc3c5c891b5
    Reviewed-on: https://code.wireshark.org/review/20960
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  99e0b39   sharkd: add tap for RTP stream analysis.
    adds  dc81822   Fix size information for sctp.sack_duplicate_tsn


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