# HG changeset patch # User Geoff Voelker # Date 896571717 0 # Node ID 83928cc3329e0e3a9fef9d51f17a13c9352bf063 # Parent e509b80a17a23292f0a491861bd7fc7ae0916a65 (sys_rename): Handle filenames with accented characters. (stat): Handle "c:/.." and "/.." specially. diff -r e509b80a17a2 -r 83928cc3329e src/w32.c --- a/src/w32.c Sat May 30 23:41:48 1998 +0000 +++ b/src/w32.c Sat May 30 23:41:57 1998 +0000 @@ -1446,6 +1446,7 @@ { char temp[MAX_PATH]; DWORD attr; + int result; /* MoveFile on Windows 95 doesn't correctly change the short file name alias in a number of circumstances (it is not easy to predict when @@ -1484,9 +1485,13 @@ name - we will end up deleting the file we are trying to rename! */ newname = map_w32_filename (newname, NULL); - /* TODO: Use GetInformationByHandle (on NT) to ensure newname and temp - do not refer to the same file, eg. through share aliases. */ - if (stricmp (newname, temp) != 0 + /* Suggested by Pekka Pirila : stricmp does not + handle accented characters correctly, so comparing filenames will + accidentally delete these files. Instead, do the rename first; + newname will not be deleted if successful or if errno == EACCES. + In this case, delete the file explicitly. */ + result = rename (temp, newname); + if (result < 0 && errno == EACCES && (attr = GetFileAttributes (newname)) != -1 && (attr & FILE_ATTRIBUTE_DIRECTORY) == 0) { @@ -1494,7 +1499,7 @@ _unlink (newname); } - return rename (temp, newname); + return result; } int @@ -1628,7 +1633,7 @@ int stat (const char * path, struct stat * buf) { - char * name; + char *name, *r; WIN32_FIND_DATA wfd; HANDLE fh; DWORD fake_inode; @@ -1650,6 +1655,13 @@ return -1; } + /* If name is "c:/.." or "/.." then stat "c:/" or "/". */ + r = IS_DEVICE_SEP (name[1]) ? &name[2] : name; + if (IS_DIRECTORY_SEP (r[0]) && r[1] == '.' && r[2] == '.' && r[3] == '\0') + { + r[1] = r[2] = '\0'; + } + /* Remove trailing directory separator, unless name is the root directory of a drive or UNC volume in which case ensure there is a trailing separator. */