Wireshark-commits: [Wireshark-commits] master e2e20c0: ssl: dtls: Fix small memory leaks
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 22 Jul 2018 12:31:48 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e2e20c0dbbddf76f714c34e32ba4821aeab572eb
Submitter: Peter Wu (peter@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

e2e20c0 by Vasil Velichkov (vvvelichkov@xxxxxxxxx):

    ssl: dtls: Fix small memory leaks
    
    Use g_strsplit/g_strfreev instead of wmem_strsplit/wmem_free because in
    wmem_strutil.h the wmem_strsplit is documented not to be used with a
    NULL allocator.
    
    The wmem_free does not free the string elements in contrast to g_strfreev.
    
    Change-Id: Ia207d5df5b8b7edd4ba77071292761ae4819fb12
    Reviewed-on: https://code.wireshark.org/review/28796
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  60d7288   [Automatic update for 2018-07-22]
    adds  e2e20c0   ssl: dtls: Fix small memory leaks


Summary of changes:
 epan/dissectors/packet-dtls.c | 8 ++++----
 epan/dissectors/packet-ssl.c  | 8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)