From c32849c5bc5de00a65e542ebeaa20e77c59b3357 Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Thu, 9 Oct 2008 09:43:02 +0000 Subject: [PATCH] * path.cc (path_conv::is_binary): Always check for GetBinaryType. * syscalls.cc (rename): Fix test for just changing case of name on case insensitive filesystems. --- winsup/cygwin/ChangeLog | 6 ++++++ winsup/cygwin/path.cc | 7 ++----- winsup/cygwin/syscalls.cc | 3 ++- 3 files changed, 10 insertions(+), 6 deletions(-) diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 0c8e76ee3..30932645c 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,9 @@ +2008-10-09 Corinna Vinschen + + * path.cc (path_conv::is_binary): Always check for GetBinaryType. + * syscalls.cc (rename): Fix test for just changing case of name on + case insensitive filesystems. + 2008-10-09 Corinna Vinschen * path.cc (symlink_info::check_shortcut): Don't check for executability diff --git a/winsup/cygwin/path.cc b/winsup/cygwin/path.cc index 47513ae88..9b85a93d9 100644 --- a/winsup/cygwin/path.cc +++ b/winsup/cygwin/path.cc @@ -1299,11 +1299,8 @@ path_conv::is_binary () tmp_pathbuf tp; PWCHAR bintest = tp.w_get (); DWORD bin; - /* Do NOT check for .exe suffix, otherwise rename(2) misbehaves - when renaming files to existing executables with omitted suffix. - strip(1) is a candidate uncovering wrong behaviour here. */ - return exec_state () == is_executable - || GetBinaryTypeW (get_wide_win32_path (bintest), &bin); + + return GetBinaryTypeW (get_wide_win32_path (bintest), &bin); } /* Normalize a Win32 path. diff --git a/winsup/cygwin/syscalls.cc b/winsup/cygwin/syscalls.cc index 1bbf570e3..68bfa93d3 100644 --- a/winsup/cygwin/syscalls.cc +++ b/winsup/cygwin/syscalls.cc @@ -1608,7 +1608,8 @@ rename (const char *oldpath, const char *newpath) just a request to change the case of the filename. By simply setting the file attributes to INVALID_FILE_ATTRIBUTES (which translates to "file doesn't exist"), all later tests are skipped. */ - if (oldpc.objcaseinsensitive () && newpc.exists () && equal_path) + if (oldpc.objcaseinsensitive () && newpc.exists () && equal_path + && old_explicit_suffix == new_explicit_suffix) { if (RtlEqualUnicodeString (oldpc.get_nt_native_path (), newpc.get_nt_native_path (),