Wireshark-commits: [Wireshark-commits] master bcaf997: wsutil: introduce ws_pipe_close() helper and
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 01 Nov 2018 06:26:19 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bcaf997f8af1ac0c718140977d1b4f98a3321533
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

bcaf997 by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    wsutil: introduce ws_pipe_close() helper and use it to terminate mmdbresolve
    
    We were not calling TerminateProcess() to stop mmdbresolve.Exe process on
    Windows.
    
    Bug: 15248
    Change-Id: Ic90cf438a8003a6fefb023b7056984681ce09b46
    Reviewed-on: https://code.wireshark.org/review/30449
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  54b58c5   mmdb: do not lock the pipe mutex recursively
     add  bcaf997   wsutil: introduce ws_pipe_close() helper and use it to terminate mmdbresolve


Summary of changes:
 debian/libwsutil0.symbols |  1 +
 epan/maxmind_db.c         |  3 +--
 extcap.c                  |  5 +----
 wsutil/ws_pipe.c          | 12 ++++++++++++
 wsutil/ws_pipe.h          |  6 ++++++
 5 files changed, 21 insertions(+), 6 deletions(-)