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