[PATCH 4/4] ntdll: Fix read_directory_getattrlist() to get the name of a symlink rather than its target, but still detect if the symlink is broken.
Ken Thomases
ken at codeweavers.com
Wed May 20 14:52:03 CDT 2015
On May 20, 2015, at 2:42 PM, Matteo Bruni <matteo.mystral at gmail.com> wrote:
> 2015-05-20 20:40 GMT+02:00 Ken Thomases <ken at codeweavers.com>:
>> ---
>> dlls/ntdll/directory.c | 18 ++++++++++++++++--
>> 1 file changed, 16 insertions(+), 2 deletions(-)
>
> Not sure it matters in practice but it might be better to check for
> HAVE_SYS_VNODE_H in both the definition of and the call to
> read_directory_getattrlist().
Hmm. I think it's OK. The code is currently guarded by HAVE_GETATTRLIST. Since ATTR_CMN_OBJTYPE is defined in terms of the type codes, they must be available.
Including sys/vnode.h is just the way to pull in the definitions in the current SDK. It's conceivable, for example, that the types may migrate to sys/attr.h and the include of sys/vnode would be unnecessary/pointless. In other words, what's important is that the type codes are defined, not necessarily that we can include this particular header.
-Ken
More information about the wine-devel
mailing list