Wireshark-commits: [Wireshark-commits] master-1.12 d454818: dwarf: fix bug found by MSVC2013 Code A
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, 8 Feb 2015 03:52:26 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d4548183135850046502d3e4da62f85d0523c44c
Submitter: Bill Meier (wmeier@xxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

d454818 by Bill Meier (wmeier@xxxxxxxxxxx):

    dwarf: fix bug found by MSVC2013 Code Analysis
    
    The following doesn't quite do what it might seem to be doing:
    
    *value |= (byte & 0x7F) << shift;  //guint64 *value // guint8 byte
    
    The warning from MSVC2013:
     Arithmetic overflow:  32-bit value is shifted, then cast to 64-bit
     value. Results might not be an expected value
    
    Change-Id: I06e196559ec0e84da77d8866355ae7f86ba43f73
    Reviewed-on: https://code.wireshark.org/review/7021
    Reviewed-by: Bill Meier <wmeier@xxxxxxxxxxx>
    

Actions performed:

    from  5a03e8d   MIPv6: Mobility Header Link Layer Address is parsed incorrectly
    adds  d454818   dwarf: fix bug found by MSVC2013 Code Analysis


Summary of changes:
 epan/dwarf.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)