Wireshark-commits: [Wireshark-commits] master c11a912: icmp, wimax: Fix undefined shift

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 15 Jun 2015 10:55:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c11a912b6ff39f45eae3b3b8075e87b5c96d1a20
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c11a912 by Peter Wu (peter@xxxxxxxxxxxxx):

    icmp,wimax: Fix undefined shift
    
    Fixes these warnings reported by Undefined Behavior Sanitizer (UBSan)
    while running the test suite:
    
        icmp: left shift of 55099 by 16 places cannot be represented in type 'int'
        wimax: left shift of 1 by 31 places cannot be represented in type 'int'
    
    Change-Id: I72913a901b61033098750da9c8f1617b055999a1
    Reviewed-on: https://code.wireshark.org/review/8913
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  7102a06   Qt/Bluetooth: Add Devices dialogue
    adds  c11a912   icmp,wimax: Fix undefined shift


Summary of changes:
 epan/dissectors/packet-icmp.c        |   10 ++++------
 plugins/wimaxasncp/wimaxasncp_dict.l |    2 +-
 2 files changed, 5 insertions(+), 7 deletions(-)