[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

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(-)