Wireshark-commits: [Wireshark-commits] master 50639e9: dumpcap: fix memleak if multi_files_on is en
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: Thu, 24 Jan 2019 21:25:50 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=50639e924314b07755b45318cff96537f920fb92
Submitter: "Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

50639e9 by Peter Wu (peter@xxxxxxxxxxxxx):

    dumpcap: fix memleak if multi_files_on is enabled
    
    Capture tests fail under ASAN due to leaking capture_opts->save_file.
    Since v2.9.0rc0-1493-g787d61c0a4, capture_opts_cleanup takes care of
    freeing "save_file", so avoid clearing the pointer.
    
    Change-Id: Ice90efe0959cc8016f47db20970bd2397909e28d
    Reviewed-on: https://code.wireshark.org/review/31727
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  f51821a   ZigBee: Changed file offset from BASE_HEX to BASE_DEC
     add  50639e9   dumpcap: fix memleak if multi_files_on is enabled


Summary of changes:
 dumpcap.c | 2 --
 1 file changed, 2 deletions(-)