Wireshark · Wireshark-commits: [Wireshark-commits] master 1b574b9: capchild: remove double free (found by clang).

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1b574b907f8ce97b00d402e0374b8943c83543d4
Submitter: Peter Wu (peter@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

1b574b9 by Dario Lombardo (lomato@xxxxxxxxx):

    capchild: remove double free (found by clang).
    
    Now the callers are responsible for deallocating argv: not doing it
    can lead to memleaks.
    
    Change-Id: I45dc0826c0430e38426eb64555664892744aa2d5
    Reviewed-on: https://code.wireshark.org/review/25369
    Petri-Dish: Dario Lombardo <lomato@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  8bc1c6e   Add DIAG_OFF_FLEX and DIAG_ON_FLEX for use in Flex scanners.
    adds  1b574b9   capchild: remove double free (found by clang).


Summary of changes:
 capchild/capture_sync.c | 99 +++++++++++++++----------------------------------
 1 file changed, 29 insertions(+), 70 deletions(-)