Wireshark-commits: [Wireshark-commits] master e489df5: Don't assert that values fit in 32 bits if t

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: Fri, 19 Sep 2014 21:54:05 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e489df5024acc86b0bd3c9f41844b06ff7560d24
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

e489df5 by Michael Mann (mmann78@xxxxxxxxxxxx):

    Don't assert that values fit in 32 bits if the NDR64 flag is set.
    
    For now, this just pacifies fuzz-testing.  If real world examples have this, there needs to be a drastic overhaul to support fields that could be either 32 or 64-bit values.
    
    Bug:9329
    Change-Id: I3e28808ca0291868a5f84258b0ee1e2a922703c2
    Reviewed-on: https://code.wireshark.org/review/4189
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  9fce9ba   Bluetooth: ACL/AVDTP: Fix ACL session handling
    adds  e489df5   Don't assert that values fit in 32 bits if the NDR64 flag is set.


Summary of changes:
 epan/dissectors/packet-dcerpc-ndr.c    |    6 +--
 epan/dissectors/packet-dcerpc.c        |   67 +++++++++++++++++++++++---------
 epan/dissectors/packet-dcerpc.h        |    2 +-
 epan/dissectors/packet-dcom-oxid.c     |    4 +-
 plugins/profinet/packet-dcerpc-pn-io.c |   13 +++++--
 5 files changed, 65 insertions(+), 27 deletions(-)