Wireshark-commits: [Wireshark-commits] master-2.4 c766c63: DIS: fix Modulation paremter length
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: Sun, 18 Feb 2018 13:00:48 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c766c6392490d94a7e15bb5b6294bf1e91e927e8
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-2.4
Repository: wireshark

Commits:

c766c63 by Uli Heilmeier (uh@xxxxxxxxxxxx):

    DIS: fix Modulation paremter length
    
    According to https://www.sisostds.org/DigitalLibrary.aspx?Command=Core_Download&EntryId=43048
    table 5.2.1 modulation paremter is 8 bytes long (and not 16).
    
    Bug: 14441
    Change-Id: I5d43411124e554721cfaa6c33561a04c91688aee
    Reviewed-on: https://code.wireshark.org/review/25866
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit bee04353b91cbec90c9377f74ceeefa374757c65)
    Reviewed-on: https://code.wireshark.org/review/25869
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  d24db8d   S7comm: fix range check to prevent infinite loop when upper bound is 255
    adds  c766c63   DIS: fix Modulation paremter length


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