diff options
| author | Corinna Vinschen <corinna@vinschen.de> | 2008-07-02 14:00:27 +0000 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2008-07-02 14:00:27 +0000 |
| commit | bb82bbe6ff1e97b1e2baba9436676aa9406fcef3 (patch) | |
| tree | 196d546c8176c195ed6f0204191397961160125b /winsup/cygwin/path.cc | |
| parent | 278101de7f2a6d1276c2f670a5ae30425cc5d33b (diff) | |
| download | cygnal-bb82bbe6ff1e97b1e2baba9436676aa9406fcef3.tar.gz cygnal-bb82bbe6ff1e97b1e2baba9436676aa9406fcef3.tar.bz2 cygnal-bb82bbe6ff1e97b1e2baba9436676aa9406fcef3.zip | |
* path.cc (path_conv::is_binary): Fix test. Add comment.
Diffstat (limited to 'winsup/cygwin/path.cc')
| -rw-r--r-- | winsup/cygwin/path.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/winsup/cygwin/path.cc b/winsup/cygwin/path.cc index 0eeee8cd8..c69f9d88e 100644 --- a/winsup/cygwin/path.cc +++ b/winsup/cygwin/path.cc @@ -1314,9 +1314,11 @@ 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 - && RtlEqualUnicodePathSuffix (get_nt_native_path (), L".exe", TRUE) - && GetBinaryTypeW (get_wide_win32_path (bintest), &bin); + || GetBinaryTypeW (get_wide_win32_path (bintest), &bin); } /* Normalize a Win32 path. |