Re: [PATCH] ntdll: Don't append directory entries if the filename length is less than 1.
"Lei Zhang" <thestig(a)google.com> writes:
I saw the following in a +file trace a user sent me:
trace:file:FindNextFileW returning L"" (L"")
I tried fixing FindNextFileW(), but that patch didn't get accepted. I'm guessing we should fix the problem in NtQueryDirectoryFile() so nobody who call it will get back an empty file name.
This should never happen. You should figure out where the empty name comes from and fix the caller. -- Alexandre Julliard julliard(a)winehq.org
On Thu, Oct 30, 2008 at 4:08 AM, Alexandre Julliard <julliard(a)winehq.org> wrote:
"Lei Zhang" <thestig(a)google.com> writes:
I saw the following in a +file trace a user sent me:
trace:file:FindNextFileW returning L"" (L"")
I tried fixing FindNextFileW(), but that patch didn't get accepted. I'm guessing we should fix the problem in NtQueryDirectoryFile() so nobody who call it will get back an empty file name.
This should never happen. You should figure out where the empty name comes from and fix the caller.
It's probably because we didn't check d_ino, patch sent. Though wouldn't it be good to have some sanity checking in append_entry(), since all the read_directory helper functions use it?
"Lei Zhang" <thestig(a)google.com> writes:
On Thu, Oct 30, 2008 at 4:08 AM, Alexandre Julliard <julliard(a)winehq.org> wrote:
"Lei Zhang" <thestig(a)google.com> writes:
I saw the following in a +file trace a user sent me:
trace:file:FindNextFileW returning L"" (L"")
I tried fixing FindNextFileW(), but that patch didn't get accepted. I'm guessing we should fix the problem in NtQueryDirectoryFile() so nobody who call it will get back an empty file name.
This should never happen. You should figure out where the empty name comes from and fix the caller.
It's probably because we didn't check d_ino, patch sent. Though wouldn't it be good to have some sanity checking in append_entry(), since all the read_directory helper functions use it?
I'm not a big fan of the belt-and-suspenders approach. If the callers are behaving correctly there's no need for an extra check, and if they aren't then we shouldn't attempt to cope with their bugs. Besides, there's no guarantee that invalid entries would be empty, they may also contain garbage, and we can't fix that in append_entry. -- Alexandre Julliard julliard(a)winehq.org
participants (2)
-
Alexandre Julliard -
Lei Zhang