Wireshark · Wireshark-commits: [Wireshark-commits] master a051d5d: wsutil: Refactor WIN32 ws_pipe_wait_for_pipe()

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a051d5d869e1932a718eb43bc2089e88cadd30e3
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

a051d5d by Tomasz Moń (desowin@xxxxxxxxx):

    wsutil: Refactor WIN32 ws_pipe_wait_for_pipe()
    
    The ws_pipe_wait_for_pipe() implementation had multiple issues:
      * Use auto-reset events with ConnectNamedPipe (should be manual-reset)
      * Leaking event handles
      * Not checking return value from CreateEvent()
      * Waiting on closed handles
    
    This change fixes all the above mentioned issues.
    
    Bug: 15696
    Change-Id: Ia0c389a902655f85eccb0c59288b4a7d49da48c9
    Reviewed-on: https://code.wireshark.org/review/32896
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Tomasz Moń <desowin@xxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  324710e   ISAKMP: Adding some payload types (RFC6407)
     add  a051d5d   wsutil: Refactor WIN32 ws_pipe_wait_for_pipe()


Summary of changes:
 wsutil/ws_pipe.c | 129 ++++++++++++++++++++++++++++++++-----------------------
 1 file changed, 75 insertions(+), 54 deletions(-)