win pipe: race condition on queueing uv_pipe_zero_readfile_thread_proc

fixes a race condition where if uv_close was called shortly after uv_read_start or a successful read and before the uv_pipe_zero_readfile_thread_proc thread started, that thread would try to read the mutex object after it had already been deleted

Author: vtjnash

Diff for: src/win/pipe.c

@@ -1114,7 +1114,6 @@ static DWORD WINAPI uv_pipe_zero_readfile_thread_proc(void* parameter) {
   assert(handle->type == UV_NAMED_PIPE);
 
   if (handle->flags & UV_HANDLE_PIPE_READ_CANCELABLE) {
-    uv_mutex_lock(m); /* mutex controls *setting* of readfile_thread */
     if (DuplicateHandle(GetCurrentProcess(), GetCurrentThread(),
                         GetCurrentProcess(), &hThread,
                         0, TRUE, DUPLICATE_SAME_ACCESS)) {
@@ -1248,9 +1247,13 @@ static void uv_pipe_queue_read(uv_loop_t* loop, uv_pipe_t* handle) {
   req = &handle->read_req;
 
   if (handle->flags & UV_HANDLE_NON_OVERLAPPED_PIPE) {
+    if (handle->flags & UV_HANDLE_PIPE_READ_CANCELABLE)
+        uv_mutex_lock(&handle->pipe.conn.readfile_mutex); /* mutex controls *setting* of readfile_thread */
     if (!QueueUserWorkItem(&uv_pipe_zero_readfile_thread_proc,
                            req,
                            WT_EXECUTELONGFUNCTION)) {
+      if (handle->flags & UV_HANDLE_PIPE_READ_CANCELABLE)
+          uv_mutex_unlock(&handle->pipe.conn.readfile_mutex);
       /* Make this req pending reporting an error. */
       SET_REQ_ERROR(req, GetLastError());
       goto error;