[PATCH] mkdir: always check-for-existence
Corinna Vinschen
corinna-cygwin@cygwin.com
Tue Jun 4 07:42:00 GMT 2019
Hi Ben,
On Jun 3 22:07, Ben wrote:
> When creating a directory which already exists, NtCreateFile will correctly
> return 'STATUS_OBJECT_NAME_COLLISION'.
>
> However when creating a directory and all its parents a normal use would
> be to start with mkdir(‘/cygdrive/c’) which translates to ‘C:\’ for which
> it'll
> instead return ‘STATUS_ACCESS_DENIED’.
>
> So we better check for existence prior to calling NtCreateFile.
> ---
> winsup/cygwin/dir.cc | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/winsup/cygwin/dir.cc b/winsup/cygwin/dir.cc
> index f43eae461..b757851d5 100644
> --- a/winsup/cygwin/dir.cc
> +++ b/winsup/cygwin/dir.cc
> @@ -331,8 +331,10 @@ mkdir (const char *dir, mode_t mode)
> debug_printf ("got %d error from build_fh_name", fh->error ());
> set_errno (fh->error ());
> }
> + else if (fh->exists ())
> + set_errno (EEXIST);
> else if (has_dot_last_component (dir, true))
> - set_errno (fh->exists () ? EEXIST : ENOENT);
> + set_errno (ENOENT);
> else if (!fh->mkdir (mode))
> res = 0;
> delete fh;
>
> --
> 2.21.0
I was just trying to apply your patch but it fails to apply cleanly.
Can you please check your indentation? The `else' lines are indented
more than the lines in between and TABs are missing.
Maybe your MUA breaks the output? If all else fails, you could attach
your patch as plain/text attachement to your mail, usually that's left
alone by the MUA.
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Maintainer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20190604/deed1d9c/attachment.sig>
More information about the Cygwin-patches
mailing list