Fix error handling in the directory iterator on Windows. Fix incorrect timeout setting for no-space retries in the changelog. Thanks to Vlad Khorsun.

dyemanov · dyemanov · commit 08437bf4c040 · 2025-09-29T16:37:55.000+03:00
diff --git a/src/common/os/win32/path_utils.cpp b/src/common/os/win32/path_utils.cpp
@@ -55,7 +55,8 @@ void Win32DirIterator::init()
 
 	if (dir == INVALID_HANDLE_VALUE)
 	{
-		if (GetLastError() != ERROR_FILE_NOT_FOUND)
+		auto const last_error = GetLastError();
+		if (last_error != ERROR_FILE_NOT_FOUND && last_error != ERROR_PATH_NOT_FOUND)
 			system_call_failed::raise("FindFirstFile");
 
 		dir = 0;
diff --git a/src/jrd/replication/ChangeLog.cpp b/src/jrd/replication/ChangeLog.cpp
@@ -66,9 +66,9 @@ using namespace Replication;
 
 namespace
 {
-	const unsigned FLUSH_WAIT_INTERVAL = 1; // milliseconds
+	const unsigned FLUSH_WAIT_INTERVAL = 1; 	// milliseconds
 
-	const unsigned NO_SPACE_TIMEOUT = 10;	// seconds
+	const unsigned NO_SPACE_TIMEOUT = 10000;	// milliseconds
 	const unsigned NO_SPACE_RETRIES = 6;		// up to one minute
 
 	const unsigned COPY_BLOCK_SIZE = 64 * 1024; // 64 KB

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,8 @@ void Win32DirIterator::init()`
`55`	`55`
`56`	`56`	`if (dir == INVALID_HANDLE_VALUE)`
`57`	`57`	`{`
`58`		`- if (GetLastError() != ERROR_FILE_NOT_FOUND)`
	`58`	`+ auto const last_error = GetLastError();`
	`59`	`+ if (last_error != ERROR_FILE_NOT_FOUND && last_error != ERROR_PATH_NOT_FOUND)`
`59`	`60`	`system_call_failed::raise("FindFirstFile");`
`60`	`61`
`61`	`62`	`dir = 0;`
Original file line number	Diff line number	Diff line change
`@@ -66,9 +66,9 @@ using namespace Replication;`
`66`	`66`
`67`	`67`	`namespace`
`68`	`68`	`{`
`69`		`- const unsigned FLUSH_WAIT_INTERVAL = 1; // milliseconds`
	`69`	`+ const unsigned FLUSH_WAIT_INTERVAL = 1; // milliseconds`
`70`	`70`
`71`		`- const unsigned NO_SPACE_TIMEOUT = 10; // seconds`
	`71`	`+ const unsigned NO_SPACE_TIMEOUT = 10000; // milliseconds`
`72`	`72`	`const unsigned NO_SPACE_RETRIES = 6; // up to one minute`
`73`	`73`
`74`	`74`	`const unsigned COPY_BLOCK_SIZE = 64 * 1024; // 64 KB`