diff options
author | Peter Wu <peter@lekensteyn.nl> | 2019-01-25 17:26:18 +0100 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2019-01-26 07:43:10 +0000 |
commit | ae6b585d518c56c5daa0243796c6321643e68ed7 (patch) | |
tree | 9ba4e967286df880dad4b98bb45e98b111e74d74 /epan | |
parent | 0b632861e21222513fac38487a56e442a4aac2fb (diff) |
ISAKMP: fix memory leaks for IKEv1 decryption
Fixes leaks of allocations from dissect_key_exch as detected by ASAN
while running the following tests:
test_ikev1_simultaneous
test_ikev1_unencrypted
test_text2pcap_ikev1_certs_pcap
test_ikev1_certs
Change-Id: Ifc102539efadd33d1b9d9921bcdbb35dfd31927f
Fixes: acfe071eb6 ("Add decryption support.")
Reviewed-on: https://code.wireshark.org/review/31740
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-isakmp.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/epan/dissectors/packet-isakmp.c b/epan/dissectors/packet-isakmp.c index 5bbc8e2f7f..130fe4295c 100644 --- a/epan/dissectors/packet-isakmp.c +++ b/epan/dissectors/packet-isakmp.c @@ -5792,6 +5792,8 @@ free_cookie_value(gpointer value) { decrypt_data_t *decr = (decrypt_data_t *)value; + g_free(decr->gi); + g_free(decr->gr); g_hash_table_destroy(decr->iv_hash); g_slice_free1(sizeof(decrypt_data_t), decr); } |