Additional error checks for issuing the windows.appendAtomically warning (#4528)

dscho · Git for Windows Build Agent · commit cf96e472d659 · 2023-08-07T22:21:19.000Z
Another (hopefully clean) PR for showing the error warning about atomic
append on windows after failure on APFS, which returns EBADF not EINVAL.

Signed-off-by: David Lomas &lt;dl3@pale-eds.co.uk&gt;
diff --git a/compat/mingw.c b/compat/mingw.c
@@ -949,7 +949,7 @@ ssize_t mingw_write(int fd, const void *buf, size_t len)
 {
 	ssize_t result = write(fd, buf, len);
 
-	if (result < 0 && errno == EINVAL && buf) {
+	if (result < 0 && (errno == EINVAL || errno == EBADF) && buf) {
 		/* check if fd is a pipe */
 		HANDLE h = (HANDLE) _get_osfhandle(fd);
 		if (GetFileType(h) == FILE_TYPE_PIPE)

Original file line number	Diff line number	Diff line change
`@@ -949,7 +949,7 @@ ssize_t mingw_write(int fd, const void *buf, size_t len)`
`949`	`949`	`{`
`950`	`950`	`ssize_t result = write(fd, buf, len);`
`951`	`951`
`952`		`- if (result < 0 && errno == EINVAL && buf) {`
	`952`	`+ if (result < 0 && (errno == EINVAL \|\| errno == EBADF) && buf) {`
`953`	`953`	`/* check if fd is a pipe */`
`954`	`954`	`HANDLE h = (HANDLE) _get_osfhandle(fd);`
`955`	`955`	`if (GetFileType(h) == FILE_TYPE_PIPE)`