Wireshark-commits: [Wireshark-commits] master-2.4 970ee89: 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 14:49:25 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=970ee892ff5d43f6d90a549a0103f1b352cd31a1
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.4
Repository: wireshark

Commits:

970ee89 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>
    Reviewed-on: https://code.wireshark.org/review/28799
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    

Actions performed:

    from  85660cb   [Automatic update for 2018-07-22]
    adds  970ee89   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(-)