Re: [PATCH] Cygwin: fhandler_fifo::raw_read: handle STATUS_PENDING

2021-11-23 Thread Takashi Yano
On Tue, 23 Nov 2021 15:59:05 -0500 Ken Brown wrote: > Patch attached. Takashi, since you wrote the analogous patch for pipes, > could > you take a look? This one also LGTM. -- Takashi Yano

Re: [PATCH] Cygwin: fhandler_pipe::raw_read: fix handle leak

2021-11-23 Thread Takashi Yano
On Tue, 23 Nov 2021 11:22:25 -0500 Ken Brown wrote: > Patch attached. LGTM. Thanks for fixing this. -- Takashi Yano

[PATCH] Cygwin: fhandler_fifo::raw_read: handle STATUS_PENDING

2021-11-23 Thread Ken Brown
Patch attached. Takashi, since you wrote the analogous patch for pipes, could you take a look? Thanks. KenFrom 4f47e64b11ed8d47c62fa89e9b971f44b7e9ab75 Mon Sep 17 00:00:00 2001 From: Ken Brown Date: Tue, 23 Nov 2021 11:40:56 -0500 Subject: [PATCH] Cygwin: fhandler_fifo::raw_read: handle

[PATCH] Cygwin: fhandler_pipe::raw_read: fix handle leak

2021-11-23 Thread Ken Brown
Patch attached.From 6d34b62cb8e192071e193516c23419854c3b4127 Mon Sep 17 00:00:00 2001 From: Ken Brown Date: Tue, 23 Nov 2021 10:13:43 -0500 Subject: [PATCH] Cygwin: fhandler_pipe::raw_read: fix handle leak Slightly rearrange the code to avoid returning without closing the event handle. ---