From: Dmitry Timoshkov Subject: Re: [PATCH] kernel32: CreateDirectory shouldn't return ERROR_ACCESS_DENIED for the root of the drive. Message-Id: <20181211203029.d3762dae4ee03b8ea1f8ebfe@baikal.ru> Date: Tue, 11 Dec 2018 20:30:29 +0300 In-Reply-To: <87ftv4dln2.fsf@winehq.org> References: <20181211130332.df4244c0451dcacab0fe4803@baikal.ru> <87sgz4dvei.fsf@winehq.org> <20181211201105.76ef4080e7f8efe0f2fa7289@baikal.ru> <87ftv4dln2.fsf@winehq.org> Alexandre Julliard wrote: > >> > According to the testbot results CreateDirectory("C:\\", NULL) fails > >> > with ERROR_ACCESS_DENIED for not administrators. However with UAC enabled > >> > and not and administrator account I get ERROR_ALREADY_EXISTS in that case > >> > with Windows 7 64-bit running on real hardware. Moreover, Wine doesn't > >> > really perform any access checks in that case and blindly assumes that > >> > returning STATUS_ACCESS_DENIED is correct behaviour for the drive's root: > >> > dlls/ntdll/directory.c,lookup_unix_name(). > >> > > >> > This patch fixes an application that can't find its data files because > >> > after it receives ERROR_ACCESS_DENIED it stops further directory traversing. > >> > >> What app is that? Does it fail on Windows when not administrator? > > > > The application doesn't fail on Windows, and as I mentioned above under > > a not administrator account and UAC enabled I don't get ERROR_ACCESS_DENIED > > error with the tests included in the patch. > > Your tests get ERROR_ACCESS_DENIED on every single testbot vm except > w8adm, so that's not very convincing... Is the app going to fail on all > these vms? I'd guess if the CreateDirectory("C:\") returns ERROR_ACCESS_DENIED then the app would fail. It's not clear how the VMs are configured and why I don't get ERROR_ACCESS_DENIED on real hardware under a not admin account, but it should be pretty obvious that since Wine doesn't perform any real administrator access checks, and if it would the checks should be done on the server side, ntdll checks shouldn't return access denied error. -- Dmitry.