"Lei Zhang" thestig@google.com writes:
On Thu, Oct 30, 2008 at 4:08 AM, Alexandre Julliard julliard@winehq.org wrote:
"Lei Zhang" thestig@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.