mingw: avoid inifinite loop in rename()

dscho · dscho · commit 1a3e0073f775 · 2017-10-19T17:11:44.000+02:00
We have this loop where we try to remove the read-only attribute when rename() fails and try again. If it fails again, let's not try to remove the read-only attribute and try *again*. This fixes #1299 Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
diff --git a/compat/mingw.c b/compat/mingw.c
@@ -2422,7 +2422,8 @@ int mingw_rename(const char *pold, const char *pnew)
 		return -1;
 	}
 
-	if ((attrs = GetFileAttributesW(wpnew)) != INVALID_FILE_ATTRIBUTES) {
+	if (attrs == INVALID_FILE_ATTRIBUTES &&
+	    (attrs = GetFileAttributesW(wpnew)) != INVALID_FILE_ATTRIBUTES) {
 		if (attrs & FILE_ATTRIBUTE_DIRECTORY) {
 			DWORD attrsold = GetFileAttributesW(wpold);
 			if (attrsold == INVALID_FILE_ATTRIBUTES ||

Original file line number	Diff line number	Diff line change
`@@ -2422,7 +2422,8 @@ int mingw_rename(const char pold, const char pnew)`
`2422`	`2422`	`return -1;`
`2423`	`2423`	`}`
`2424`	`2424`
`2425`		`- if ((attrs = GetFileAttributesW(wpnew)) != INVALID_FILE_ATTRIBUTES) {`
	`2425`	`+ if (attrs == INVALID_FILE_ATTRIBUTES &&`
	`2426`	`+ (attrs = GetFileAttributesW(wpnew)) != INVALID_FILE_ATTRIBUTES) {`
`2426`	`2427`	`if (attrs & FILE_ATTRIBUTE_DIRECTORY) {`
`2427`	`2428`	`DWORD attrsold = GetFileAttributesW(wpold);`
`2428`	`2429`	`if (attrsold == INVALID_FILE_ATTRIBUTES \|\|`