Wireshark-commits: [Wireshark-commits] master 897c750: Clean up some memory leaks in wsutil/filesys
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 19 Feb 2017 01:27:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=897c75012792cbfdfd275f56b9b4c4418c7354ad
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

897c750 by Michael Mann (mmann78@xxxxxxxxxxxx):

    Clean up some memory leaks in wsutil/filesystem.c
    
    The fixes involve functions always returning a string that needs to be
    freed by the caller.  That way the static string pointers within the functions
    don't end up leaking at program end.
    
    Change-Id: I35a8dc37536c2927d88f33217887f62efafcc8ca
    Reviewed-on: https://code.wireshark.org/review/20175
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  7e70311   Fix ieee80211_tag_ssid
    adds  897c750   Clean up some memory leaks in wsutil/filesystem.c


Summary of changes:
 ui/profile.c             |    6 ++-
 ui/qt/profile_dialog.cpp |    7 +++-
 wsutil/filesystem.c      |   97 ++++++++++++++++++++++++----------------------
 wsutil/filesystem.h      |    6 ++-
 4 files changed, 64 insertions(+), 52 deletions(-)