URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=52910f4e3fa7d771a671cd30bd5320d70aee3eec
Submitter: Anders Broman (a.broma...@gmail.com)
Changed: branch: master
Repository: wireshark

Commits:

52910f4 by Peter Wu (pe...@lekensteyn.nl):

    isakmp: fix type confusion
    
    Caught by ASAN with WIRESHARK_DEBUG_WMEM_OVERRIDE=simple set.
    
    Manually inspection of all type casts to decrypt_data_t and
    ikev2_decrypt_data_t showed no other users that do not check
    isakmp_version first.
    
    Change-Id: If889afff85a20e31222d33cbea8db3a91a77f389
    Reviewed-on: https://code.wireshark.org/review/17246
    Petri-Dish: Peter Wu <pe...@lekensteyn.nl>
    Tested-by: Petri Dish Buildbot <buildbot-no-re...@wireshark.org>
    Reviewed-by: MichaƂ Skalski <mskalsk...@gmail.com>
    Reviewed-by: Anders Broman <a.broma...@gmail.com>
    

Actions performed:

    from  bee0cd7   See if *this* convinces the compiler that valuelen and 
value will be set.
    adds  52910f4   isakmp: fix type confusion


Summary of changes:
 epan/dissectors/packet-isakmp.c |   24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)
___________________________________________________________________________
Sent via:    Wireshark-commits mailing list <wireshark-commits@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
             mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

Reply via email to