Commit 53d8a25dab5ddcea16ac70cdcdf28cb3e4944cbb | zig

author	Jonathan Marler <johnnymarler@gmail.com> 2022-02-05 18:59:09-07:00
committer	Jonathan Marler <johnnymarler@gmail.com> 2022-02-06 18:05:20-07:00
log	53d8a25dab5ddcea16ac70cdcdf28cb3e4944cbb
tree	15fd1fa33b9657dc8ed2eaa233333c28551b6a31
parent	adc9a282d8b3cbe58e07c965fe40fb1dd8666bd7

child_process: collectOutputWindows handle broken_pipe from ReadFile

This was found on a user's machine when calling "git" as a child process from msys.  Instead of getting BROKEN_PIPE on GetOverlappedREsult, it would occur on ReadFile which would then cause the function to hang because the async operation was never started.

1 files changed, 22 insertions(+), 6 deletions(-)

lib
- std
  - child_process.zig

lib/std/child_process.zig+22-6

 @@ -277,10 +277,19 @@ pub const ChildProcess = struct {
             const new_capacity = std.math.min(outs[i].items.len + bump_amt, max_output_bytes);
             try outs[i].ensureTotalCapacity(new_capacity);
             const buf = outs[i].unusedCapacitySlice();
-            _ = windows.kernel32.ReadFile(handles[i], buf.ptr, math.cast(u32, buf.len) catch maxInt(u32), null, &overlapped[i]);
+            const read_result = windows.kernel32.ReadFile(handles[i], buf.ptr, math.cast(u32, buf.len) catch maxInt(u32), null, &overlapped[i]);
-            wait_objects[wait_object_count] = handles[i];
+            std.debug.assert(read_result == 0);
-            wait_object_count += 1;
+            switch (windows.kernel32.GetLastError()) {
+                .IO_PENDING => {
+                    wait_objects[wait_object_count] = handles[i];
+                    wait_object_count += 1;
+                },
+                .BROKEN_PIPE => {}, // don't add to the wait_objects list
+                else => |err| return windows.unexpectedError(err),
+            }
         }
+        if (wait_object_count == 0)
+            return;
         while (true) {
             const status = windows.kernel32.WaitForMultipleObjects(wait_object_count, &wait_objects, 0, windows.INFINITE);
 @@ -320,9 +329,16 @@ pub const ChildProcess = struct {
             try outs[i].ensureTotalCapacity(new_capacity);
             const buf = outs[i].unusedCapacitySlice();
             if (buf.len == 0) return if (i == 0) error.StdoutStreamTooLong else error.StderrStreamTooLong;
-            _ = windows.kernel32.ReadFile(handles[i], buf.ptr, math.cast(u32, buf.len) catch maxInt(u32), null, &overlapped[i]);
+            const read_result = windows.kernel32.ReadFile(handles[i], buf.ptr, math.cast(u32, buf.len) catch maxInt(u32), null, &overlapped[i]);
-            wait_objects[wait_object_count] = handles[i];
+            std.debug.assert(read_result == 0);
-            wait_object_count += 1;
+            switch (windows.kernel32.GetLastError()) {
+                .IO_PENDING => {
+                    wait_objects[wait_object_count] = handles[i];
+                    wait_object_count += 1;
+                },
+                .BROKEN_PIPE => {}, // don't add to the wait_objects list
+                else => |err| return windows.unexpectedError(err),
+            }
         }
     }