Wireshark-commits: [Wireshark-commits] master cfe7dc8: 80211 utils: free nl messages after use
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, 5 Jul 2015 17:30:31 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cfe7dc8bab00b80478418416de5354996fea3d3b
Submitter: Evan Huus (eapache@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

cfe7dc8 by Evan Huus (eapache@xxxxxxxxx):

    80211 utils: free nl messages after use
    
    Valgrind picked this up as a very large memory leak on systems with libnl,
    since the GUI polls this code regularly and was leaking several nl messages
    each time.
    
    Change-Id: Ie6b32e094d90183a16fb187adea430c4b43c208c
    Reviewed-on: https://code.wireshark.org/review/9502
    Petri-Dish: Evan Huus <eapache@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    

Actions performed:

    from  c00e469   Fix Windows CMake generated installer to include snmp mibs
    adds  cfe7dc8   80211 utils: free nl messages after use


Summary of changes:
 caputils/ws80211_utils.c |   17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)