Wireshark-commits: [Wireshark-commits] master dd1245f: ws_pipe: fix memory leaks in spawn arguments

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 23 Apr 2019 23:18:35 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=dd1245f5be20e10f8ef917b20c302129b505cd97
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

dd1245f by Peter Wu (peter@xxxxxxxxxxxxx):

    ws_pipe: fix memory leaks in spawn arguments handling
    
    On Windows, ws_pipe_spawn_sync always leaks 'winargs', and leaks 'argv'
    on some error paths. Fix these and refactor the common argument parsing
    functionality to reduce duplication of functionality.
    
    Change-Id: I8fa5ca45aec20b53f6fa243b0dd07241a345f7ab
    Reviewed-on: https://code.wireshark.org/review/32932
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Tomasz Moń <desowin@xxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  4dfa358   nas-5gs: Fix ecoding of Authorised QoS rules
     add  dd1245f   ws_pipe: fix memory leaks in spawn arguments handling


Summary of changes:
 wsutil/ws_pipe.c | 182 ++++++++++++++++++++++++++++++-------------------------
 wsutil/ws_pipe.h |   1 +
 2 files changed, 102 insertions(+), 81 deletions(-)